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
Not Found
Not Found
Not Found

Submitted

NFT Preview Card Component using flex-box

@Clare1950

Desktop design screenshot for the NFT preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


Hi all,

I would love some feedback on: How can I make the mobile view better? How can I structure the html better for accessability?

Thanks for your help! Clare

Community feedback

@Clare1950

Posted

Hi,

Thanks so much for your feedback really appreciate it as I am just starting out!

I put the pseudo class as :active rather than :hover which I think is causing the issues. I will fix this.

Thanks for the accessibility tips!

Clare

0
P
David Turner 4,110

@brodiewebdt

Posted

I think the mobile view looks fine on this. The layout looks good on this. The minimum and maximum sizes are only about 20px difference. Your hover effects aren't working correctly. The overlay only shows up on the image when you click on it. When you click on the Equilibrium text the overlay shows on the image. Another thing you should do is put your stylesheet below any other links in the head of the document. I put mine as Reset styles, Google fonts, Font Awesome and lastly Custom styles.

Wrap your container div in a Main tag and it will clear the accessibility warnings.

Download AXE DevTools and you can clear accessibility warnings while you code. https://www.deque.com/axe/devtools/

Hope this helps.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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