Skip to content
Submitted about 3 years ago

HTML, CSS, Flexbox, Javascript

jhuanjulio•230
@jhuanjulio
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 love to hear comments from you guys

Community feedback

  • Dharmik_487•1,740
    @Dharmik48Posted about 3 years ago

    Hey👋,

    I have a few suggestions for you:

    • The height of the card is way to too much than the design and doesn't look good, so decrease it.
    • Also at big screen, the share pop up that appears when clicked on the share button is oddly placed.
    • And also change the height of the body to max-height: 100vh rather than 200vh. And then to center the card you can use display: flex; place-items: center;.

    Apart from these your solution is really good! Keep it up👍

    Marked as helpful
    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