Skip to content
Submitted about 3 years ago

grid, flex, position relative/absolute, z-index

César Palma•425
@xsrpm
Preview site (opens in new tab)View code (opens in new tab)
This is a solution for...Article preview component
  • HTML
  • CSS
  • JS
1newbie

Design comparison

SolutionDesign

Solution retrospective

Would you have a feedback for me about the button when I clicked? Any comment is welcome.

Community feedback

  • Vinay Puppal•160
    @vinaypuppalPosted about 3 years ago

    Hey, overall your solution looks good. Few suggestions

    1. Make sure vertical align the card.
    2. Add some box-shadow to the card
    3. Check some HTML and ACCESSIBILITY issues reported here
    4. Add padding around the card for small screens(i.e below 330px), at present it's touching the edges of the screen
    Marked as helpful
    1
  • Fluffy Kas•7,715
    @FluffyKasPosted about 3 years ago

    Hey, your solution looks good! Just make sure you add alts for your images. They need to be there, even if your images are decorative (in this case they can be left empty).

    Marked as helpful
    1

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