Responsive single price grid using flexbox and grid
Design comparison
Solution retrospective
This is my solution for this challenge. Any suggestions or comments appreciated
Community feedback
- @danielmrz-devPosted 11 months ago
Hello @belkysupreme22!
Your project looks nice!
I have a few tips for you to improve it:
-
You can replace your
h2
withh1
. The<h1>
to<h6>
tags are used to define HTML headings.<h1>
defines the most important heading.<h6>
defines the least important heading. Note: Only use one<h1>
per page - this should represent the main heading/subject for the whole page. Also, do not skip heading levels - start with<h1>
, then use<h2>
, and so on. -
Still about semantic HTML code: You can also replace your
div.container
withmain.container
. This won't change anything visually, but it makes your HTML code more semantic and improves SEO optimization as well as the accessibility of your project. -
Since the Signup button is a clickable element, it's a good practice to add
cursor: pointer;
to it. It gives the user a visual indication that it's clickable.
I hope it helps!
Other than that, you did a good job!
2 -
Please log in to post a comment
Log in with GitHubJoin 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