Design comparison
SolutionDesign
Solution retrospective
done with flex-box am waiting feedback
Community feedback
- @Zy8712Posted 12 months ago
Your site and code look great. Good job!!!
Some things I would change with regards to your website:
- you didn't use a
<h1>
tag on anything and skipped straight to<h2>
. You should change your headers so that there is at least one<h1>
tag as headers are meant to be used in order with out any skips. You can always customize the font-size in css with thefont-size
property. - you should add a hover effect to your button using the
:hover
feature so your site has more interactivity - for the third box ("Why Us"), you should use an unordered list to list everything out because right now when the screen size changes there are some instances where the text is just one big paragraph. An unordered list is something like this:
<ul> <li>Item 1</li> <li>Item 2</li> <li>Item 3</li> </ul>
Hope you find this feedback useful 👍
Marked as helpful0 - you didn't use a
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