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 Card Preview using HTML CSS and Javascript

Gabi 280

@Gabocz

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


I used Javascript to create the hover effect on the image of the cube. Could this be done without using JS? I am beginner so I don't know. :-) Also, any suggestions on how to get rid of the flashing when I move the mouse on the cube image? Thank you!

Community feedback

Rio Cantre 9,690

@RioCantre

Posted

Hello there! Nice job in completing this project. Regarding your solution, I would like to recommend the following for you…

  • Add the hover state of the design. For the hero image , refer it with this one Overlay in image
  • Wrap the whole content with specific tag like main and attribution with footer tag. For HTML structures, refer it with this one Semantics
  • Include description with the alt in image tags. Like this line <img id="mainTheme" src="./images/image-equilibrium.jpg" alt="">
  • Import the attribution style in CSS file and remove the style tag
  • Remove unnecessary code to keep it clean
  • Add display: flex; and justify-content: center; in .info span:nth-of-type(1) and .info span:nth-of-type(2) to align the icons with the text. Add margin or padding between them as well.

Above all, the project is done well. Keep up the good work and Hope this is helpful!

Marked as helpful

2

Gabi 280

@Gabocz

Posted

@RioCantre Thank you for the suggestions! I will try to re-work the project using them!

2
Naveen Gumaste 10,460

@NaveenGumaste

Posted

Hay ! Gabi Good Job on challenge

These below mentioned tricks will help you remove any Accessibility Issues

-> Add Main tag after body <main class="container"></main>

-> Learn more on accessibility issues

If this comment helps you then pls mark it as helpful!

Have a good day and keep coding 👍!

Marked as helpful

1

@WesleyKaihara

Posted

Hello, maybe to solve the problem of the flickering image you could put in #overlay{ transition:all .5s; }

https://developer.mozilla.org/pt-BR/docs/Web/CSS/transition

Marked as helpful

1

Gabi 280

@Gabocz

Posted

@WesleyKaihara Thank you, I will try that!

0
Hexer 3,660

@Phalcin

Posted

Great work overall man. Keep it up

1

Gabi 280

@Gabocz

Posted

@Phalcin Thank you man! I am a complete beginner so any incouragement is welcome. :-)

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