@Titli007
Submitted
@CosteaAlin93
@Titli007
Submitted
@CosteaAlin93
Posted
Hi Ananya :) Very nice work on the design. I would have a few extra hints:
Other than that, all seems fine, congrats for the solution. Let me know if my hints are not clear :)
All the best,
Alin
Marked as helpful
@vishal-singh5128
Submitted
Please provide any suggestions for further learning. I found difficulties when implementing localStorage, maybe there is a suggestion. Let me know if there are any errors, even better if provide a solution. Thank You
@CosteaAlin93
Posted
Hi Vishal, There could be a better way to do this. First of all, let's define some cases:
clicking submit while no rating is selected >> this case should show an error message or some hint to the user to click a rating before submitting. To do this, above the submit button, create a paragraph element which will contain something like 'Please select a rating!' , and style it to be hidden (display: none;
) and only show it when the button is clicked without a rating being selected
selecting a rating and pressing submit >> this should not point to to index2.html. Rather, use the same trick as above. Consider your content on index.html as the 'rating view' and the content on index2.html as the 'thank you' view. Combine the HTML from the both files in a single one. Now, if you have a rating selected, clicking submit should hide your 'ratingBox' element and in his place, you should display the 'thankYouState ratingBox' element.
and one more thing: clicking submit on your solution will give a rating by default. If you fix the first case above, this situation will disappear.
Marked as helpful
@FerneyMontoya6
Submitted
If someone have feedback for this, I'd like to know it. I'll be greatful 😊
@CosteaAlin93
Posted
Hi Ferney! Awesome design!! At a very close inspection, I would modify just a few minor things to make it 100% pixel perfect:
Once again, congrats for the solution. Hope my hints make sense and are on the point. Drop me a message if anything is not clear.
All the best,
Alin
@meliodas-png
Submitted
@CosteaAlin93
Posted
Hello Meliodas!
Congrats for your work! The design is almost perfect, except for some very minor things:
Keep up the good work, let me know if you have and doubts about my answer :)
Marked as helpful
@jeff-dev0
Submitted
@CosteaAlin93
Posted
Hello Jafar ! Nice work :) A few remarks:
Keep up the good work, let me know if you have and doubts about my answer :)
Marked as helpful
@Pulkit-s21
Submitted
Did this challenge earlier with vanilla HTML CSS and now I completed it using react tailwind
@CosteaAlin93
Posted
Hi Pulkit! Overall, it looks good. A few hints for the design:
On the Mobile view:
Considering you are using Tailwind for this, my advice would be:
Keep the solutions coming! Planning to do your approach once I start learning a bit or React!
Marked as helpful
@IsaacLezama
Submitted
Hi, I made the solution for this challenge but I'm not very good with media querys and responsive designs, so it would be really appreciated if you could make observations and comment some advice or resources, thank you all in advanced!
@CosteaAlin93
Posted
Hello Isaac!
Try the above and let me know! Hope my review helps.
Best wishes,
Alin
Marked as helpful
@replacethrill
Submitted
I accept any kind of help, thank you for your attention. HTML/CSS.
@CosteaAlin93
Posted
Hello Tales :) Your design is very good. What you could do to make it 1:1 with the target:
Keep up the good work!
Marked as helpful