Skip to content
Submitted about 1 month ago

Review of the Material Covered SCSS JS custom fonts

Kristina•280
@frontend-en
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Interactive rating component
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

I am most proud of implementing a responsive layout using SCSS and Flexbox. Next time, I would optimize my JavaScript code for better performance.

What challenges did you encounter, and how did you overcome them?

I encountered challenges with aligning elements using Flexbox, but I overcame them by extensively referring to the documentation and experimenting with different flex properties.

What specific areas of your project would you like help with?

I would like help with optimizing JavaScript performance and improving SCSS structure for better maintainability.

Community feedback

  • Grace•31,230
    @grace-snowPosted about 1 month ago

    I'm afraid the html needs to change in this. You need to use a group of radio inputs, not a load of buttons. Once that's changed you'll have to change the css and js as well, but the good news is the js should become much simpler. This won't need any click listeners at all, only one submit listener on the form.

    Marked as helpful
    0

Please log in to post a comment

Log in with GitHub

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