Vanilla JS/CSS

Community feedback

• 

  Krushna Sinnarkar•1,100

  @krushnasinnarkarPosted 4 months ago

  Hi @pertrai1,

  Congratulations on successfully completing the challenge!

  Your solution looks nice though there are a couple of things you can improve which I hope will be helpful!

  You have used width: 325px; for the article element. It would be better if you use max-width: 325px; instead. When the screen size is reduced to less than 325px, the card will go out of view if you use a fixed width. By using max-width, the card's width will adjust automatically if the screen size is smaller than the maximum width.

  article {
      max-width: 325px;
  }
  

  I hope you find this helpful, and I would greatly appreciate it if you could mark my comment as helpful if it was.

  Feel free to reach out if you have more questions or need further assistance.

  Happy coding!

  0