Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Article Preview Component Master(HTML, CSS and JAVASCRIPT)

@waynerafael15

Desktop design screenshot for the Article preview component coding challenge

This is a solution for...

  • HTML
  • CSS
  • JS
1newbie
View challenge

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-k-phillips

Posted

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

@waynerafael15

Posted

@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
Discord logo

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