Skip to content
Submitted over 3 years ago

NFT Card Preview using HTML CSS and Javascript

Gabi•280
@Gabocz
A solution to the NFT preview card component challenge
Preview (opens in new tab)Code (opens in new tab)

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!

Code

Please log in to post a comment

Log in

Community feedback

  • Rio Cantre•9,650
    @RioCantre

    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
  • Naveen Gumaste•10,420
    @NaveenGumaste

    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
  • Wesley kaihara•60
    @WesleyKaihara

    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
  • Hexer•3,620
    @EmmanuelHexer

    Great work overall man. Keep it up

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