Submitted 4 months ago
Product Card Component Using HTML, CSS, Flebox.
@FabianAM07
Design comparison
SolutionDesign
Solution retrospective
What are you most proud of, and what would you do differently next time?
Next time, I will research and practice more to make a better responsive card.
What challenges did you encounter, and how did you overcome them?Adjust the @mediaqueries to adapt the screen for mobile use was difficult.
What specific areas of your project would you like help with?If you have tips for this code let me know, thanks! =)
Community feedback
- @mkborisPosted 4 months ago
Great work FabianAM07, some things you should look at
- Responsive images are handled using the
picture
element. Using CSS to hide and show images to display differently on various screen sizes is not part of the best practices. - The
a
tag is supposed to be abutton
look up when to use a button and an anchor element. Anchors are for navigation while buttons are for events like toggling content. - Font-size should be written in
rem
not px. This article explains it better Why font-size must NEVER be in pixels. - Media queries should be defined in
rem
orem
not px. - Consider using a modern CSS reset at the start of the styles in every project. Like this one Modern CSS Reset.
Hope this helps.
Marked as helpful1@FabianAM07Posted 4 months ago@mkboris Thank you very much for the tips and articles, they have very good information to make better web pages.
0 - Responsive images are handled using 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