Design comparison
SolutionDesign
Solution retrospective
Any feedback is highly appreciated :)
Community feedback
- @NaveenGumastePosted almost 3 years ago
Hay ! Good Job Ali
These below mentioned tricks will help you remove any Accessibility Issues
-> Add
Main
tag after body like it should be your container. For 1st heading orh1
tag, use header tag and then inside the header put yourh1
orh2
etc . But use header tag only once in main heading element.Keep up the good work!
Marked as helpful1 - @RioCantrePosted almost 3 years ago
Hello there! Awesome work with this one. Viewing your solution, I would suggest the following for you...
- Adjust the breakpoint of the media query into
620px
- Adjust the padding into
padding: 39% 37% 33% 37%;
inside the.eye-overlay
rule set - Wrap the
container
with specific tag likemain
for readability - Include description for the
alt
inimg
tag and should not be empty
Overall you did well and Keep it up!
Marked as helpful1 - Adjust the breakpoint of the media query into
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