Design comparison
SolutionDesign
Solution retrospective
Had fun with this project, good learning experience. I set a <div> for every button to get the alignment. Is this the usual approach to aligning buttons?
Community feedback
- Account deleted
Hello 👋 @diazeddym
This looks good! 👌 A few things you can do to improve it a little bit:
- It's good to change the master container tag to the
<main>
to clearly idenfity and structure the main content section of the webpage, enhancing accessibility, SEO, and code readability. - Maybe you know this, but instead of the
<button>
element, it should be<a>
, since they are links. - Try to be a little more descriptive for the alt attribute.
- Try to always use rem instead of px, which will provide better scalability and responsiveness across devices. Also, you can use the max-width property instead of width for better flexibility and responsiveness.
Keep it up! 😍🚀
Marked as helpful2 - It's good to change the master container tag to the
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