Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Oops! You need to be logged in to do that
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Interactive rating component using HTML, CSS & JS

Andrew 90

@L1m1tz

Desktop design screenshot for the Interactive rating component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View Challenge

Design Comparison

SolutionDesign

Report

0

Accessibility Issues

0

HTML Issues

View Report

Andrew’s questions for the community

This is one of my first js projects

I've done most of the challenge just missing the right colors for the inactive buttons and some styling for the response.

any tips on ways to improve will be greatly appreciated.

Community Feedback

@Iamweird2

Posted

Hi, nice work so far.

  1. To be very sure of the colors, you can open the design file on your browser, then open developer option by (control shift i) and search around for a dropper. you can also search online how to find a color dropper for your web browser.

  2. You can also center the details on the submit page to improve the aesthetics.

  3. Endeavor not to use style elements by tag name or with id, using classes all through saves time debugging and prevent issues with specificity

  4. Using static units such as pixels for font size is also not advisable since page sizes vary, specify for font size in relation to the browser window size using rem or em.

I hope this helps. Have a nice day.

Marked as helpful

1

Grace 21,200

@grace-snow

Posted

Hi

Really important to use a form element and radio inputs in a fieldset on this. It can't be a load of buttons like that.

Other things that's causing problems on mobile with content being cut off is the height 100vh. Never limit height like that when there is written content inside - you need to use min-height

Hope thats helpful,along with the comments above

Marked as helpful

1


Andrew 90

@L1m1tz

Posted

@grace-snow Thank you for the feedback. I limit my body's height so that I can get the container to the center, is there a different way to the container in the center without leaving a huge gap at the bottom?

0

Grace 21,200

@grace-snow

Posted

@L1m1tz use min height like I said

Marked as helpful

0

Please focus on giving high-quality, helpful feedback and answering any questions L1m1tz might have. Here are some key points to consider:

  • Does the solution include semantic HTML?
  • Is it accessible, and what improvements could be made?
  • Does the layout look good on a range of screen sizes?
  • Is the code well-structured, readable, and reusable?

You can read our community guidelines if you're unsure what to post.

Slack logo

Join our Slack community

Join over 100,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!