Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Oops! You need to be logged in to do that
Not Found
Not Found

Submitted

Responsive card using flexbox and grid.

@felipe-miranda-marreiros

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View Challenge

Design Comparison

SolutionDesign

Report

3

Accessibility Issues

1

HTML Issues

View Report

Felipe Miranda Marreiros’s questions for the community

It was really fun building this card. This is my first project without tutorial hell.

Community Feedback

Aakash Verma 6,810

@skyv26

Posted

Hi! Felipe, Really nice job. I checked your design and its responsiveness. I found no issues in your work excepts the report issues. In order to solve report issues do as below:

  1. Add alt text as I saw in your report
<div class="main-img-box">
          <img src="./images/image-equilibrium.jpg" alt="" class="main-img-equilibrium">
          <div class="hover-img-icon">
            <img src="./images/icon-view.svg" alt="More info icon">
          </div>
        </div>

as you can see above main-img-equillibrium image have no alt text. Add alt to solve issue.

  1. Wrap your section code inside the <main> tag and it will solve all your accessibility issues.

I hope it might helpful to you.

Best Wishes

1


@felipe-miranda-marreiros

Posted

@skyv26 Thanks for the tips. I'll fix them.

0

P

@Eric-Ferole

Posted

Hi Felipe,

Great job overall. Here's my two cents :

  • You could put a display: block; on your img tag to remove the small gap at the bottom when overing on it
  • You could add à cursor: pointer; to your div.hover-img-icon

I hope this comment helps you.

Keep up !

Marked as helpful

1


@felipe-miranda-marreiros

Posted

@Eric-Ferole thanks for the tips. The display: block worked perfectly!

1

Please focus on giving high-quality, helpful feedback and answering any questions felipe-miranda-marreiros might have. Here are some key points to consider:

  • Does the solution include semantic HTML?
  • Is it accessible, and what improvements could be made?
  • Does the layout look good on a range of screen sizes?
  • Is the code well-structured, readable, and reusable?

You can read our community guidelines if you're unsure what to post.

Slack logo

Join our Slack community

Join over 100,000 people taking the challenges, talking about their code, helping each other, and chatting about all things front-end!