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

I used HTML, CSS and Javascript to complete this challenge

@Offiongwhilley

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


At first, I struggled with my javascript code. I was not sure which function to define within which. After many attempts, my logic worked.

I would like to know whether there are other ways which I can restructure my javascript code so that it's shorter and simpler to read.

Community feedback

T
Grace 29,310

@grace-snow

Posted

Hi

I notice issues with the html that need fixing on this

  • Indent your code consistently. Your code editor can even do this formatting automatically for you
  • this design shows a form with radio inputs. It should not be a div with a load of buttons inside (essential)
  • use landmark elements to wrap page content. This should be a main element for the content and a footer element for the frontend mentor attribution
  • remove all the br tags from the paragraphs. Let the max width of the container determine where lines break, never try to control it like that

Marked as helpful

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