Skip to content
Submitted over 4 years ago

I used HTML, CSS and JS.

Martins Michael•60
@McClef
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

Hey @everyone.

i woold love some feedback in this project as i know i missed out on somethings in the process of coding.

Thanks.

Community feedback

  • Ramsay Sewell•585
    @ramsaysewellPosted over 4 years ago

    Hey Martins,

    Great job on this solution, it looks great!

    Just a few tips to help work with Flexbox would be to use flex-basis instead of width. This will help deal with the wrapping of elements. There is a large number of resources online regarding flex-basis.

    I would look into using overflow: hidden; on your image container too, as this will force the image to stay in the container.

    Look into using a box-shadow too for the entire element rather than using a border. This will allow to look closer to the original design.

    Let me know if there's anything here confusing you or if you're stuck with anything.

    Thanks and happy coding!

    Ramsay

    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