Recipe Page

Community feedback

• 

  Ryan O'Hanlon•140

  @Ryan-OHanlonPosted 4 months ago

  A very strong match to the desktop design. While the code is well structured, it could do better with added comments to understand what each section of HTML or CSS is doing what.

  The color of Simple Omelette Recipe does not have any color attribute added from the style guide.

  While the site is responsive to the desktop design, the webpage does not change to match the mobile design. You need to make a second set of CSS rules for the desktop using the @media rules and apply a min-width. Then put all the desktop rules to adjust the margin and padding for inside the @media rule.

  /*Desktop CSS rules when wider than 375px*/
  @media (min-width: 376px) {
      body{
      }
  }
  

  Also very good using the margin left attribute to create space for all listed elements and the table.

  0