seyide hundeyin• 440
@SeyideHundeyin
Posted
Looks really nice and responsive!
1
Vjekoslav• 485
@Vj3ko
Posted
@SeyideHundeyin ty for feedback :)
1
Design comparison
Solution retrospective
Finished this single price grid layout, i would be really happy if someone can check it out and comment what i can do better, or to shorten code, or anything really. Until next time, keep on coding !
Community feedback
Hachi• 155
@hachi-ops
Posted
Responsiveness works perfect, well done. I would suggest to make paragraphs font size on your mobile design same size for all of your divs. Also, you forgot to add hover state to the button (see README.md file for this project).
1
Vjekoslav• 485
@Vj3ko
Posted
@hachi-ops actually i did add a hover state to the button. i didnt read readme file but i read it now, and it only says that i need to add hover state. so i picked my own color and add a little transition to it. about paragraphs , ty for that insight.
ty for the comment, i really appreciate it :)
0
Hachi• 155
@hachi-ops
Posted
@Vjekoni I just noticed that hover state! I think the reason why I didn't see is because it is almost the same green colour, only a different shade. It may be a good idea to make it completely different colour so as to make it easier for colour-blindness accessibility. I am not colour-blind but didn't see it until you said you have that hover state there.
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