@zitescody
Posted
Hello, @jacobharraldson. Wonderfully done! The structure is readable and fluid (at least to me). But we can improve the readability further! Here is how:
1.) Change that <div class="wrapper"> element to <main class="wrapper">. Whenever possible, use landmarks (<main> <section> <article> <aside> <footer> <header>) in your html to bolster accessibility and readability on your webpage. This makes it easier for screen readers to crawl through your webpage and lets those who use accessibility features to know exactly what part of the page they are on.
That's it! This simple fix and your index.html file is much more readable! Excellent job on the custom properties in CSS. It's by far my favorite feature.
Please mark this as helpful if you benefitted from my feedback. Happy coding!
Marked as helpful
@jakub-ozog
Posted
@zitescody Thank you for that! This is what I wanted to get from the feedback. Thanks once again, cheers!