so... i used JavaScript for this Challenge. To be honest, i could have usad an href to navigate to the page, but the challenge required javascript, so why not? i used onclick to hide a significant part of the tags and make other ones appears.
Yassine Afaila
@YassineafailaAll comments
- @joashnevesSubmitted over 1 year ago@YassineafailaPosted over 1 year ago
hi bro , hope you doing great :D you need to add to the buttons a cursor-pointer, and if you want also to add a function to the dismiss message button for back to the first page .
0 - @grlcodrSubmitted over 1 year ago@YassineafailaPosted over 1 year ago
hi ,the desing looks good and the code also good but i have one suggestion lines of code : =>you could use the value attribute of the button like this : {[1, 2, 3, 4, 5].map((num) => ( <button className="btn" onClick={handleClick} key={num} value={num}> {num} </button> ))} =>which affect on this: const handleClick = (e) => { setRating(e.target.value); }; and also it will be better if you learn a framework like bootstrap 5 or tailwind
Marked as helpful0 - @mohamedbntahaSubmitted over 1 year ago
It was a nice project, but I think there are a lot of useless elements
@YassineafailaPosted over 1 year agoHi bro :D, The design looks fine and yeah you have a useless element there ,You need to replace the div. Payment with button tag . good luck
0 - @graiess073029Submitted over 1 year ago
Hello Everyone. I completed this challenge almost successfully. When i deployed the application, it can't get the header svg and the fonts. So, i'm waiting for you recommendation to fix this and to perform more the app
@YassineafailaPosted over 1 year agoHi bro ;D ,The design looks great and for you question I think you need just to add background-size:cover; background-repeat:no-repeat; background-position:center; add this to your header tag and I think it will work
0 - @Japjotsingh02Submitted over 3 years ago
Any feedback and suggestions will be appreciated.
@YassineafailaPosted over 1 year agoI want to say something to you :D ,I really really thank you for code in JavaScript it's helped me a lot I swear to god thank you so much if you have time I want to talk with you.
1 - @casablancassSubmitted over 1 year ago
I still have trouble centering the main content vertically on the page
Is it really necessary to create a <div> and then put an <img> inside?
@YassineafailaPosted over 1 year agoHi, if you want to center the content you could use this: display:flex; align-items:center; justify-content:center; height:100vh; flex direction:column;
0 - @Zuzana83Submitted over 1 year ago
After first solution with Chart.js, this second attempt was done with just HTML, CSS & Javascript.
@YassineafailaPosted over 1 year agoHello, the design looks great ,I don't know if I'm right but I will recommended this method for you :in the JavaScript you can use the XMLHttpRequest() instead of using the variable to store the data from JSON file .you can look at my solution in my repo and see how i did .
0 - @juanmfretesSubmitted over 1 year ago
Hi guys. I need your help with an issue.
When you are in 'responsive state' and unselect a 'rating-item', this stays with gray background color (i.e. hover state in 'desktop state').
How can I fix this error? Thanks in advance for your help
@YassineafailaPosted over 1 year agohello,i think for the issue that you need to fix is add the hove effect on <li>or <button> in the responsive mode
0 - @AdnaanHSubmitted over 1 year ago
The project was amazing and got challenged when working with Javascript events but got a wind of it. Any help on better ways of accomplishing the same results would be appreciated?
@YassineafailaPosted over 1 year agoHello, my friend! The design is looking good. I have a few suggestions. Instead of using the <ul> element, you could use the <form> element and <input type='radio'> elements because we need to submit the rating. If you're not sure how to do this, you can check my code. Good luck!
Marked as helpful0 - @sasanqcSubmitted over 1 year ago
This was my first solution with sass. I hope get some comments . Thanks
@YassineafailaPosted over 1 year agoHi ,please how did you do :in the mobile device ,when the menu show up how did you do that darken background or shadow in the background
0 - @MelvinAguilarSubmitted almost 2 years ago
Hi there 👋, I’m Melvin and this is my solution for this challenge. 🚀
🛠️ Built With:
- TailwindCSS
- npm - prettier
Any suggestions on how I can improve and reduce unnecessary code are welcome!
Thank you. 😊✌️
@YassineafailaPosted almost 2 years agowhere did you learn Tailwind CSS?? the design looks great, keep going bro
1 - @AzkanorouziSubmitted about 2 years ago
Hey community, I have a little surprise for you 😊. Feel free to give me any suggestion or advice on how to make this better.
@YassineafailaPosted almost 2 years agothat design is unique defiantly is unique. Keep going bro
1