Skip to content
Submitted about 2 months ago

grid layouts and media queries

Emerson Lima•200
@poissonfou
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...NFT preview card component
  • HTML
  • CSS
1newbie

Design comparison

SolutionDesign

Solution retrospective

What are you most proud of, and what would you do differently next time?

.

What challenges did you encounter, and how did you overcome them?

.

What specific areas of your project would you like help with?

.

Community feedback

  • Elizabeth Shackelford•70
    @E-C-ShackelfordPosted about 2 months ago

    Hi! Great work on developing your card component! I have a few suggestions that could enhance both accessibility and clarity. First, consider using more semantic HTML elements, such as <article> instead of <div> where it makes sense. This can improve the structure of your code and make it easier for assistive technologies to understand.

    I noticed you're using relative units for padding and margins, which is great, but I recommend also using relative units for font sizes rather than pixels. This will make your card component more adaptable and accessible for users who adjust their text size.

    Take another look at your CSS for classes with many common properties. Creating a shared class for these styles can help keep your stylesheet cleaner and easier to manage.

    You've made a great start with this card component. Keep up the great work!

    0

Please log in to post a comment

Log in with GitHub

Join 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