Article preview component responsive page (HTML, CSS, JS)

Community feedback

• 

  P

  Ryan Bradley•630

  @ryanbradley-webdevPosted almost 2 years ago

  Hey there, great work on this project!

  I was just looking through your code and noticed the layout regrading the 'header', 'main', and 'footer' sections in the markup. If this were a full-page project, I probably wouldn't have this advice, but since it's an 'article preview' project I feel that it's necessary.

  The use of a 'footer' tag for the bottom portion of the card's content seems a little out of place. The footer should be the bottom-most part of the page, in your case the statement referencing the challenge and you as the author. This is a little arbitrary based on the content of the project, but in the future I'd recommend putting that content in a div or section as opposed to the footer tag.

  Like I said, it looks great! Keep up the great work!

  Marked as helpful

  0
• 

  Csaba Müller•260

  @CsaboooPosted almost 2 years ago

  Hi Ryan,

  Thank you for your feedback!

  I take it into consideration by the next project.

  Best regards,

  Csaba

  0