Submitted about 3 years ago
Single Price Component - First Time using Sass
@Gabrielbe17
Design comparison
SolutionDesign
Solution retrospective
This is my first time using sass, a quick project to get in touch with it, some practice may not be the best so I'd like to receive feedback, thanks!!
Community feedback
- @pikapikamartPosted about 3 years ago
Hey, great work on this one. Desktop layout is great, it is responsive and the mobile state looks really great as well.
Also awesome that you used sass/scss on this one!.
Some suggestions would be:
- Avoid skipping a heading level. If you use
h3
make sure thath1 and h2
are both present before it. - The price text could just use
p
tag since theMonthly Subscription
heading tag already gives an information on what the section would/could contain. - Lastly, on the
why us
section, those items should not be inside a singlep
tag because those are "list" of things on why you should choose them, so useul
on them and each item isli
on its own.
Aside from those , this looks really great.
Marked as helpful1 - Avoid skipping a heading level. If you use
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