Design comparison
Solution retrospective
Any feedback ? Just comment it below :)
Community feedback
- @brodiewebdtPosted almost 3 years ago
This looks good. You did a very good job matching the design. On your hover, add a cursor:pointer to the CSS to show that it is "clickable" even though these aren't. You want to do that for any links.
Wrap your section in a Main tag, and change the H3 to an H tag. You will have to restyle it to look the same as it does now. Every page should have an H1 tag. Do these two things and it will clear the accessibility warnings.
Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/
Hope this helps.
Marked as helpful0 - @5hraddhaPosted almost 3 years ago
Hi @GirafOnTrip, Good work!! There is a suggestion that you could implement. It always better to link stylesheets in the header of the HTML markup. You could also import fonts from Google fonts in the separate css file using
@import
rule.Thank you!!
Marked as helpful0
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