@correlucas
Posted
👾Hello Adriel Murray, Congratulations on completing this challenge!
Great start and great first solution! You’ve done really good work here putting everything together, I’ve some suggestions you can consider applying to your code:
1.To make your CSS code easier to work you can create a single class
to manage the content that is mostly the same for the 3 cards (paddings, colors, margins and etc) and another class to manage the characteristics that are different (colors and icon), this way you'll have more control over then and if you need to change something you modify only one class.
2.You’ve used <div>
to wrap the card container, in this case you need to use <main>
since this is the main block of this page.
3.Every page needs a main heading, the <h1>
to show which is the most important heading. You need to increase the headings by one level, like h1, h2, h3 to show the titles hierarchy. Remember that you cannot have more than one h1 heading.
4.The component html structure its fine, something you can do to improve the html markup its to replace some divs, note that you build all the structure using div blocks and these don’t have an particular meaning, for this reason its better you use meaningful tags like <article>
in this example, for the cards replacing the divs, this way you have a tag that describes what it's inside the this html block.
✌️ I hope this helps you and happy coding!
Marked as helpful