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 component

@manonthemon

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


First small project following the completion of a front-end bootcamp to refresh some basics. Fairly random use of bootstrap and jQuery. I'm sure the code could benefit from some refactoring, but for now I'm just happy that it works.

Community feedback

Klaudia 120

@klaudiapalubska

Posted

Good job with your project. I glanced at your code and I want get your attention to some stuff.

  • From start try to focus make all layout responsive. For example
#thankyou-card{
    width: 320px;
    padding: 3vh;
    gap: 0.5vh;
}

Its nice because you use vh/vw but also you can define max-width instead width and make design responsive for smallest devices. Moreover better is use rem instead px. Thanks to this u can save a lot of time for getting tired with set properties for diffrent size devices. You only need set (for example):

html {
  /*Now 1rem = 18px*/
  /*If you are not set base, font-size is 16px and rem is references to the font-size*/
  font-size: 18px;
}

And now you can simple change all stuff in breakpoints of media queries.

  • You didnt use media queries so now your layout just mess up.

    You really need take a look at this because is really important part of front-end. The most important rules of responsive website is:

    1. Fluid layouts - how I mention try make responsive web from start. If something doesnt work than you fix it in media queries.
    2. Responsive units - rem
    3. Flexible Images - set by %, width and max-width
    4. Media Queries - making breakpoints, you can check it the best of in internet.
  • Try not to use !important

if you really understand how cascade css works then you dont need to use this line of code. Its looks really dirty for other developers and they can think that you really dont knon whats going on in your code.

  • Better set class for all elements in css file. Its because id is for one element and class is for a plenty elements. So if you will be make more expanded project you want change all elements not one.

  • You cant permit user to go to the second part of card if user doesnt select a number.

I hope I could help. Have a nice day and more fun with new projects!

Ps. Sorry for language mistakes if there is. I hope thats all is understandable what I wrotte.

Marked as helpful

1

@manonthemon

Posted

Thanks for taking the time to comment! Yes, what you say makes sense and I'll read more on rem units and responsive design in general. @Sephiress

0

@UdbhavDalmia

Posted

Almost Same nice

1

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