Recipe page main with sass

Community feedback

• 

  P

  Alexander3717•110

  @Alexander3717

  Posted 14 days ago

  Hello, I’d like to share a few suggestions.

  I noticed you used <h3> for the recipe title and <h4> for the section titles. Using <h1> and <h2> instead would be better - it keeps the heading hierarchy (headings should increase by one level at a time) and every page should contain at least one <h1>.

  For the nutrition table you used only <td> cells, wrapping 'Calories', 'Carbs'... in <th> instead is more semantic. Also last row of the table shouldn't have a border - you can fix this by styling that row differently, for example using the :last-child pseudoclass.

  Until the screen width reaches 1440px the page stays in a mobile layout. I think enhancing responsiveness by adding another breakpoint (or making the page change gradually) would improve user experience.

  In your desktop layout the recipe card touches the top and bottom edges of the page. Giving some padding to the card container should fix this easily and make it match the design more closely.

  00