Design comparison
Solution retrospective
I know I can make the responsive menu very easy with Bootstrap, but I tried using plain JS and I'm not sure if I did it in the best way, although I'm very happy with the final result.
Community feedback
- @AdrianoEscarabotePosted almost 2 years ago
Hello Felipe Rios, 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 ana
, because in my head when a person clicks on a button written Read more, he is not confirming a form, or something like, it will be redirected to another page, to read more about!to solve this problem do this:
<a href="/">Read more</a>
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. 👍
Marked as helpful1@WolfMozart8Posted almost 2 years ago@AdrianoEscarabote that's very helpful, thank you very much!
1
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