Responsive Recipe page Using Css Flexbox

Community feedback

• 

  P

  dionysia-lemonaki•130

  @dionysia-lemonakiPosted 3 months ago

  Congratulations on completing the challenge! Your project is looking really good.

  Just a small note on the HTML: Heading levels should only increase by one, so because you use h1 for the main title, the heading level under it should be h2, so it should be <h2>Preparation time</h2>. The font-size in the design does not play a role – it is about creating a hierarchy. This article from Deque University explains this concept.

  Also, for the font-size in your CSS, it is not considered a good practice to use pixels – it is best to use rem instead. This article by Grace Snow explains more on the topic.

  Marked as helpful

  0

  ---

  

  Ahmed Elmaghraby•110

  @A-ElmaghrabyPosted 2 months ago

  @dionysia-lemonaki Thank you for your thoughtful advice and kind words about my challenge! I’ll definitely check out the articles you mentioned to deepen my understanding. Thanks again for taking the time to share this valuable feedback!

  0