Design comparison
Solution retrospective
Hello, Frontend Mentor community! I'm open for suggestions!
Community feedback
- @NaveenGumastePosted almost 3 years ago
Hay ! Good Job SMSN
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 - @abhik-bPosted almost 3 years ago
👋 Hello SMSN , Your solution is wonderful & your code looks clean as well. Below are some opinions that can be helpful:
- using
min-height : 100vh;
instead ofheight
in body makes it very responsive - adding hover states to buttons can make them look nicer
Please keep contributing this amazing solutions 👍
Marked as helpful1 - using
- @RioCantrePosted almost 3 years ago
This comment was deleted almost 3 years ago
1@lloydkunPosted almost 3 years agoI already add the effect on active event not on hover but I'm gonna take note on that. Thank you.
0
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