Design comparison
SolutionDesign
Solution retrospective
any advice on making my code a little bit shorter?!
Community feedback
- @burrijwPosted over 1 year ago
Good start!
- Check your
<link>
to the normalize.css file. There's an error: "space is not allowed". - There’s horizontal overflow at 320px wide. This is inaccessible per the WCAG 2.1.
- You have a lot of things wrapped in
<div>
tags unnecessarily. Try to only use them when you need to for styling or scripting. - You don't need that div for the line. It's possible to use a border on the element before or after it instead. Or use a pseudo-element to keep your markup clean.
Marked as helpful1@stitch-606Posted over 1 year ago@burrijw
my link is the correct file name is indeed "normalize css.css", thx for the rest tips I will avoid them next time :)0@burrijwPosted over 1 year ago@stitch-606 It may be the correct file name, but the file name should not contain spaces. The browser may understand it, but that could change in the future, other browsers might stop supporting it, etc. It'd be best practice to future-proof your code by following standards. 🤝🏻
Marked as helpful0 - Check your
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