@AdrianoEscarabote
Posted
Hi Nicholas Nall, how are you?
I really liked the result of your project, but I have some tips that I think you will enjoy:
To improve the code structure, wrap this div:
<div class="attribution">
Challenge by <a href="https://www.frontendmentor.io?ref=challenge" target="_blank">Frontend Mentor</a>.
Coded by <a href="#">Nick</a>.
</div>
with the semantic tag footer
.
change the button
tag to an a
tag, taking into account that the button is Learn more
, I imagine this is not a form, or something like that, this button would probably take us to another page, read more about! so I think the a
tag is the best choice in this case!
The rest is great!
I hope it helps... 👍
Marked as helpful