Design comparison
Solution retrospective
Welcome, I hope that you will tell me your evaluation of the site and give your rating out of 10, as this makes a difference with me very much, I leave you with the site, thank you <3
Community feedback
- @rogeriobautzPosted almost 3 years ago
You are using a span tag as a div, wrapping the music icon and a div with 2 texts. Span tags can not have divs inside. Remember to update your CSS code too since you are using it in line 71.
1 - @brodiewebdtPosted almost 3 years ago
The width of the card in the Figma file is 450px. Change you rmax-width to 450px and it will look much better. Your forgot to put the background color on the Annual Plan section.
Wrap your card div in a Main tag and change the H2 to an H1. Every page should have an H1 for accessibility reasons. You will have to re-style it to look like the design. Just change H2 in the CSS file to H1 and your good to go. Those things 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.
1
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