Recipe Page

Community feedback

• 

  hitmorecode•6,360

  @hitmorecodePosted 2 days ago

  Congratulations well done. Let me just point a few things

  • When switching to small screen size, there is a overflow. Your page is not fully adapted for small screen sizes. The overflow is being caused by display flex on the body.

  • It came to my attention that you only made use of id's and no classes. Due to the fact that you've use only id's, you tend to repeat yourself quite a lot. This can be a problem if you need to make changes. If a style is going to be used multiple times, make the use of classes instead.

  • When you have an element with an id or class you don't have to target the parent and the child. You can directly target that element.

  // Instead of doing like this
  section#card > div#img {
      display: flex;
      align-items: center;
      justify-content: center;
  }
  
  // Do it like this
  #img {
      display: flex;
      align-items: center;
      justify-content: center;
  }
  

  I hope this was helpful. Keep it up 👍👌

  Marked as helpful

  0

  ---

  

  Emilly Moitinho•40

  @emillymoitinhoPosted 2 days ago

  @hitmorecode Thank you very much for the tips

  0
• 

  Raza Siddique•60

  @razzasidPosted 2 days ago

  good job i have learned a lot from your code.

  0