Looking to hire developers?
Was it Reach challenge?
@jeremiahlateef
Posted
Hi Ayush, your code needs a little bit of tweaking. For the background, add background-repeat: no-repeat, background-size: cover. For the "Annual Plan", change it from h3 to h4. Change the font-size of the dollar figure to 14 or 13.
Nice work!
Marked as helpful
@NaveenGumaste
Posted
Hay ! Good Job Ayush Nandanwar
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 or h1 tag, use header tag and then inside the header put your h1 or h2 etc . But use header tag only once in main heading element.
-> for background img add no-repeat and background-size:cover; make background-size:conatin;
Keep up the good work!
Marked as helpful
@ayushnandanwar08
Posted
@Crazimonk Thanks for commenting, this information is very useful for me, I learned about header tag and main tag that where to use it 😀
@princeunn
Posted
hello there!.. Yes you have almost done that perfectly. The only thing it is missing is the hover effect. Try applying those and it would be perfect. Need help for hover?? you can visit this site and learn more (https://www.w3schools.com/csSref/sel_hover.asp).
Marked as helpful
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
