Design comparison
SolutionDesign
Community feedback
- @Ahmed-ElbaldPosted 10 months ago
Hi Asma,
Again, I do believe you are really talented when it comes to applying the UI. I like how it's responsive on all screen sizes. I just have some notes here that you might find useful:
- Text like '30-day, hassle-free money back guarantee' should not be wrapped in an
h2
element as it's not actually a page heading. Use a regularp
element instead as headings imply the beginning of a new section. - Try to make your classes names more descriptive so that other developers can test your code more easily. For instance, you can replace your
one
,two
andthree
classes withintro
,cta
(call to action) andoutro
or anything that is meaningful. - Again, you should've used a description list
dl
for the pricing.
Marked as helpful0@asmabladPosted 10 months agoThank you so much for your response, i have now fixed the issue.
0 - Text like '30-day, hassle-free money back guarantee' should not be wrapped in an
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