Skip to content
Submitted about 1 month ago

Pure HTML + CSS + JS solution using CSS flexbox layout

h13meyer•60
@h13meyer
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 tried to take the feedback of my first submitted solution into account: I hosted the required font directly inside the project and also used a set of CSS reset rules to faciliate development.

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

Since I worked on this challenge as part of the "Web Accessibility" learning path, I would appreciate improvement suggestions regarding that topic.

Community feedback

  • KapteynUniverse•1,600
    @KapteynUniversePosted about 1 month ago

    To answer your request; i do believe there are times for alt texts are not necessary, like here. These are for cosmetics purposes

    <img src="./images/icon-star.svg" alt="Icon Star">
<img class="thank-you-img" src="./images/illustration-thank-you.svg" alt="Illustration of mobile phone">

    if you just put alt="" i think screen readers jumps them

    Also pixels are fixed sizes and don't scale well on different devices. Instead, use rem or em, which are relative units that adjust based on user settings, making your design more flexible, responsive, and accessible.

    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