Responsive rating page with HTML, CSS and Javascript.

Community feedback

• 

  P

  visualdennis•8,375

  @visualdennissPosted over 1 year ago

  Hey Victoria,

  cool work again! I'd have few suggestions for further improvements:

  • Using width: 30vw; is not necessary, it'll only cause it to shrink too much when resizing, instead using something like width: 100%, max-width: 450px (in rem/em) is a better approach.

  • You have used height: 50vh; but in general try to avoid giving fixed heights or max-heights for text containing elements like you have for .container, even if it is in rem, because it can cause issues later on (with changing dynamic data, when the user changes font-sizes, or even browser rendering issues etc.). The container height should be determined by the height of its children elements.

  • Adding a cursor: pointer; to the all buttons would be good, indicating interactivity to the user (UX)

  • the paragraph text seems a little too cramped, increasing the line-height slightly would give them some breathing room and improve readability.

  • Font family seems to be missing for "You selected 3 out of 5"

  Hope you find this feedback helpful!

  Marked as helpful

  0

  ---

  

  Victoria Ekene Ozoanidiobi•40

  @victoriagreatPosted over 1 year ago

  @visualdenniss Thank you

  0