A Div tag as a container using Css display block
Design comparison
Solution retrospective
The whole process went well without stress, it's my first time working on front-mentor challenge and its so soft and amazing.
Community feedback
- @Cyber-ChicPosted over 1 year ago
Wonderful job completing your first challenge! ๐
I have some recommendations that may improve your site:
- Change your background color. You used a slightly different color. To do this I would change your code from:
background-color: #e7ebf9;
to
background-color: hsl(218, 44%, 22%);
-
Add a <h1> element to your code. All sites should include a minimum of one <h1> element to indicate the the most important (or highest-level) heading on a page. To fix this, you could change your <h4> element to an <h1> element.
-
Add a <main> tag around the main content on your webpage. All web pages should include one <main> tag. This is a semantic HTML element that defines the main content on a page such as primary sections, text, articles, etc.
Example:
<main> <div class='container'> <h1>Main heading text</h1> <p>paragraph text</p> </div> </main>
- Your code does not look formatted. Formatting your code can make it easier to read. You can format your code using an automatic code formatter such as Prettier ยท Opinionated Code Formatter.
I hope this is helpful! Great work & Happy Coding!
-Angie
Marked as helpful0
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