@AdrianoEscarabote
Posted
Hello maryan, how are you? I truly loved your project's outcome, however I have some advice that I hope you'll find useful:
I noticed that you used a button
in which case the best option would be an a
, because in my head when a person clicks on a button written learn more, he is not confirming a form, or something like, it will be redirected to another page, to learn more about!
to solve this problem do this:
<a href="/" class="mt-lg-4 btn3 rounded-pill">learn more</a>
I noticed that you used more than one h1
for this component, this is not considered a good practice since we should only have one h1
per page, to inform the main title!
To improve the code structure wrap this div:
<div class="attribution">
with the semantic tag footer
The remainder is excellent.
I hope it's useful. 👍