Hi, Dan, good job for finishing your challenge, that quiet a step forward. I have a few things to take note:
- at your Readme file, appreciate if you change the word and delete unnecessaries.
- problem with your accessibility, but I am not sure how it look like at mobile, only can help you with this code source below.
this is ur code below which has problem:
<div class="attribution">
Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>.
Coded by <a href="#">Your Name Here</a>.
</div>
(explanation from google) Landmarks provide a way to organize the various types of content on a page for users of assistive technologies. When content is not contained in a landmark, it will be unreachable using landmark navigation, which is an important feature provided by assistive technologies such as screen readers. Read more here: https://fae.disability.illinois.edu/rulesets/LANDMARK_2/
The most important landmark roles are main and navigation, as nearly every page will include at least those regions.
You can put
<main>
<div>
</div>
</main>
If you find my feedback helpful please mark as helpful. Thank you. Continue do the hard work. You can do it! You are awesome!