Submitted over 2 years ago

Single Price Grid using HTML CSS

Devesh• 120

@devesht21

Preview site(opens in new window)View code(opens in new window)

This is a solution for...

Single price grid component

HTML
CSS

1newbie

View challenge

Design comparison

SolutionDesign

Solution retrospective

What can I improve in terms of best practices?

Community feedback

Ibtissam• 260

@Devibtissam

Posted over 2 years ago

well done, but it seems like you forgot to work on the media query for mobile, and be aware of your paddings and margins try to make it a little bit like in design given

Devesh• 120

@devesht21

Posted over 2 years ago

@Devibtissam Will take care about that next time. Thank you for the valuable feedback!

Rio Cantre• 9,690

@RioCantre

Posted over 2 years ago

Hello there! Great work with this one. Regarding the solution you submitted, I think you should know the following notes…

Remove unnecessary code to keep it clean
Add the hover state of the button

Other than that...

The HTML structure is well organized and readable
Proper usage of semantic tags
Well done in using proper properties in styling
The project is responsive and utilized the details based on the original design

Above all, The project is awesome! Keep it up!

Devesh• 120

@devesht21

Posted over 2 years ago

@RioCantre Just forgot about the hover states😅. Will take care of that next time. Thanks for your feedback!

Please log in to post a comment

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