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 project

@llr3v0ll

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


while doing this I encountered an issue. For the radio-buttons I couldn't get the "checked" button text color to be white therefore I made the text white by default. Any idea of how that could be solved?

Community feedback

Ozzy-codes 130

@Ozzy-codes

Posted

Hi @llr3v0ll

Congrats on completing this challenge!

I wanted to bring to your attention that It appears the component doesn't quite fit in the 375px mobile view. Take a look at my forked repo for a quick suggestion!

Also very interesting use of radio inputs to toggle classes, was that inspired from MDN? I wonder how this approach takes in the scope of accessibility, any experts that can weigh in? @visualdenniss

Lastly, I did a little refactor of your JS, pulling out lines 12-14. I'm curious to hear what your objective was with utilizing the plm variable?

best,

Ozzy

1

@franekbu

Posted

good job, but your site is not responsive as it should, just use

@media and screen (max-width:value) {
body{
change what you need
}
etc
}
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