@FlaviuD94
Posted
Hi, nice to have you around! Welcome to Frontend Mentor Coding community Your solution looks great, you can still do some things to it:
- replace the <h2> containing the main title with <h1> note that this title is the main heading for this page and every page needs one h1 to show which is the most important heading. Use the sequence h1 h2 h3 h4 h5 to show the hierarchy of your titles in the level of importance, never jump a level.
- you can use the <main> instead of using <section>.
- for measurement you can use ruler tool from Photoshop.
I hope it will help you, Enjoy coding!
Marked as helpful
@MohLutfiFauzi
Posted
@FlaviuD94 At first I thought it didn't fit for the card to have h1 in it, but I realized that there was only 1 component on the page. so there really should be an h1, thanks a lot.