Submitted 10 months ago

Single Price Grid Component

Namrata Sinha• 530

@nmrtsnh

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

Hello everyone,

I am Namrata and this is my solution for the Single Price Grid Component challenge.

If you have any suggestions to improve my code, please feel free to share!

Thank you 🙏

Community feedback

Øystein Håberg• 9,430

@Islandstone89

Posted 10 months ago

HTML:

Make .container a <main>.
Headings should always be in order, so you would never start with a <h4>. Change it to a <h1>.

CSS:

Performance-wise it's better to link the fonts in the <head> of the HTML, instead of using @import.
Font-size must never be in px. Use rem instead.
height on body should be min-height.
Remove margin: 0 auto on body.
max-width on container should be in rem.

Marked as helpful

Namrata Sinha• 530

@nmrtsnh

Posted 10 months ago

@Islandstone89 Thanks for the feedback! I implemented it.

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