Skip to content
Submitted about 3 years ago

Article Preview Component Master(HTML, CSS and JAVASCRIPT)

Wayne Rafael Rodriguez Pinales•50
@waynerafael15
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

Open to all comments about my code, my goal is to improve every day more and your opinion is of great help. I await your comments.

Abierto a todos los comentarios sobre mi código, mi objetivo es mejorar cada día mas y sus opinión es de gran ayuda. Espero sus comentarios.

Community feedback

  • John Phillips•350
    @john-k-phillipsPosted about 3 years ago

    Good work on this card, a few things that I quickly noticed:

    • You have a fixed height on the card, you should avoid this; instead, make use of padding and margins to create the desired height. (This will prevent the text from wrapping outside of the container).

    • Don't forget to remove your console logs from JavaScript!

    • Within your HTML you have a list item that is commented out; perhaps remove that and push for an update.

    Marked as helpful
    1
    Wayne Rafael Rodriguez Pinales•50
    @waynerafael15Posted about 3 years ago

    @john-k-phillips Thank you very much, I will take that into account, make the modifications in the original work. I really appreciate your comment.

    I will be able to make your suggestions in practice from now on.

    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