Design comparison
SolutionDesign
Solution retrospective
Feedback will be highly appreciated
Community feedback
- @anoshaahmedPosted almost 3 years ago
To get rid of the accessibility/HTML issues shown in your Report:
- wrap everything in your body in
<main>
OR use semantic tags OR giverole=""
to the direct children of your<body>
... Click here to read more - start your headings with
<h1>
, and move up by one level each time <button>
should have anaria-label
... Click here to read more
Good job! :)
Marked as helpful1 - wrap everything in your body in
- @elroytoscanoPosted almost 3 years ago
Everything is perfectly done, from the design to the implementation of the logic.
So as to zoom in, we've got to use the "+"/"-" buttons at the bottom right which is functioning as requirement. You may improve the intuitiveness by using the mousewheel to zoom in/out.
Marked as helpful1
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