Skip to content
OverviewSolutions
2
Comments
4

Lozie

@LozPilafas70 points

I’m a mysterious individual who has yet to fill out my bio. One thing’s for certain: I love writing front-end code!

Latest solutions

Latest comments

  • Lozie•70
    @LozPilafas

    i think its really good 😊 !

    One small detail, maybe use cursor:pointer on the button !

  • Lozie•70
    @LozPilafas

    Hey!

    I think its really good , you could improve it to match the brief by making it mobile friendly:

    correct media query to snap when the width is too small correct edge rounding when stacked

    (created a pull request to see the suggested code in github)

    peace

  • Lozie•70
    @LozPilafas

    Hey! I think its really good!

    I also liked the fade hover effect.

    Things that you could implement to make it better: *Validate that the user has picked a number before submitting, you could disable the button until a number has been selected (as if you click without it shows 'you have selected undefined from 5').

    peace

    Marked as helpful
  • Lozie•70
    @LozPilafas

    I think it looks great! Only thing i would change to match the brief a bit better and also make the text readable is to change the font color of the light/grey text to some shade darker to make it more readable

    Marked as helpful