Design comparison
SolutionDesign
Solution retrospective
any review would be appreciated
Community feedback
- @fernandolapazPosted over 1 year ago
Hi π, perhaps some of this may interest you:
HTML π§±, ACCESSIBILITY β:
- Every page should have an
<h1>
to improve user experience and because it is one of the most important elements when it comes to SEO. Maybe 'Equilibrium #3429' in this case.
- The 'image-equilibrium' is a meaningful image and in case the user can't see it, the
alt
text should give a description.
CSS π¨:
- The page content could be centered using Grid or Flexbox. For example as follows:
body { min-height: 100vh; display: grid; place-content: center; }
- When there is a hover state over an element it means that it is interactive, so there must be an interactive element around it (like a link or a button).
So, we should use a link or a button for the image (depending on what happened when clicking on it).
Also 'Equilibrium #3429' and 'Jules Wyvern' should be wrapped with the
<a>
tag.
Please let me know if you want more info on any of these topics or disagree with something. I hope itβs useful π
Regards,
Marked as helpful0 - Every page should have an
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