Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Interactive rating element

kmnkat 30

@kmnkat

Desktop design screenshot for the Interactive rating component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


  1. Is that responsive enough?
  2. Code refactoring - what can I do better?
  3. Are my JS function well written? Can I do something better?
  4. I decided to add classes (with new CSS style) of components for page transition and change the appearance of clicked button. Is that a good solution? Are there any other options (except of adding inline HTML style)?

Community feedback

Mehemmed77 100

@Mehemmed77

Posted

  1. and also 2) It is responsive but not in devices that have 420px width or lower.your #rating-section div overflows Solution:Try to add max-width or give it width with percentage and don't give fixed width or heights just don't do it.Give it max-width or width with percentage.I hope this is useful; https://www.youtube.com/watch?v=VQraviuwbzU&t=694s watch this video at 4:47 He talks about it.

I liked your JS logic it is really good.You approached it differently than me and i liked it!

Marked as helpful

0

kmnkat 30

@kmnkat

Posted

@Mehemmed77 Thanks for your comment :) I am going to fix it. I like the video you sent - I think that responsiveness is still hard for me but I hope this advice will help :)

0
Mehemmed77 100

@Mehemmed77

Posted

@kmnkat Follow that guy on youtube he is so good

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord