Skip to content
Submitted over 2 years ago

Single-Page Application utilizing Javascript

Cole Kamgar•10
@PeoShmeo
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

How can I improve my code syntax?

How can I improve my HTML and CSS formatting?

Do I have any weird redundancies?

I didn't use 'flex box' because I had no idea what that was. Should I have learned and used it in this scenario?

Community feedback

  • Serhii Orlenko•345
    @grifanoPosted over 2 years ago

    Hi, Looks good. I see that you making accents on JS, and this is looking good. Unfortunately, I don’t have much experience in JS good practice. My suggestion is just to add some transition to make the hover looks more pleasant 😉 Also, there are some issues with Accessibility, you can check them. About Flexbox, yes it needs to know, as there are basics for making layout in our time. Also, I recommend learning Grid CSS too. Off-course if you didn’t plan to dive deeper into the frontend, and make focus on programming or even switch to the backend, in this case, you need to know the basics of making the layout. So you can build some fast front for your needs. Good luck 👍

    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