Mobile First Product preview card component use flexbox
Design comparison
Solution retrospective
This is My First Frontend Mentor Project please correct my mistake give your feedback Thank you
Community feedback
- @MelvinAguilarPosted almost 2 years ago
Hello there π. Good job on completing the challenge !
I have some suggestions about your code that might interest you.
HTML π:
- Use the
<main>
tag to wrap all the main content of the page instead of the<div>
tag. With this semantic element you can improve the accessibility of your page.
- Use the
<footer>
tag to wrap the footer of the page instead of the<div class="attribution">
. The<footer>
element contains information about the author of the page, the copyright, and other legal information.
-
The
alt
attribute is used to provide a text description of the image which is useful for screen reader users, assistive technology users, and search engine optimization. Improve thealt
attribute to the<img>
tag of the product.If you want to learn more about the
alt
attribute, you can read this article. π.
CSS π¨:
- Instead of using pixels in font-size (
--Font-size: 14px;
), use relative units likeem
orrem
. The font-size in absolute units like pixels does not scale with the user's browser settings. This can cause accessibility issues for users who have set their browser to use a larger font size. You can read more about this here π.
-
Use
min-height: 100vh
instead ofheight: 100vh
. Theheight
property will not work if the content of the page grows beyond the height of the viewport.
-
There is no need to repeat some properties when using media query.
Example:
.container { display: flex; justify-content: center; align-items: center; height: 100vh; background-color: var(--Cream); } @media (min-width: 799px){ .container { /* All of this is repeated, it doesn't change anything */ /* display: flex; */ /* justify-content: center; */ /* align-items: center; */ /* height: 100vh; */ /* background-color: var(--Cream); */ }
And:
.flex { display: flex; flex-direction: column; } @media (min-width: 799px) { .flex { /* display: flex; */ flex-direction: row; }}
I hope you find it useful! π Above all, the solution you submitted is great!
Happy coding!
Marked as helpful0 - Use the
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