Skip to content
Submitted almost 3 years ago

NTF card- tailwind CLI

Panagiotis Panagiotakopoulos•415
@PanuGr
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

Hi. I started to learn tailwind css a couple of days ago. If you can give me some feedback on smart practices etc, it'd be great.

I used the "grid place-content-center h-screen" classes on my body tag to center my content but it didn't work. So I hard-coded it with a style-tag ( body{min-height:100vh;} ) Any ideas why tailwind class wasn't working?

Community feedback

  • PhoenixDev22•16,950
    @PhoenixDev22Posted almost 3 years ago

    Hello @PanuGr,

    I recommend for any decorative images, each img tag should have empty alt="" ( as you did )and aria-hidden="true" attributes to make all web assistive technologies such as screen reader ignore those images in( icon-ethereum, icon-clock ).

    Overall , your solution is good . Hopefully this feedback helps

    1
    Panagiotis Panagiotakopoulos•415
    @PanuGrPosted almost 3 years ago

    @PhoenixDev22 omg 😂 i was so focused on tailwind that i completely forgot the basics. Thanks for the feedback!

    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