Recipe Page

Community feedback

• 

  Younes Iddahamou Idrissi•60

  @iddahadev

  Posted 11 months ago

  Does the solution include semantic HTML?

  Overral it does, but a few things need to be fixed:

  • There are h2 and h4 but no h1 and h3.
  • The main title should be h1.
  • The preparation title should be h2.
  • The span to make the text bold should be a strong or b.

  Is it accessible, and what improvements could be made?

  • The alt for the image should be human readable.
  • The table row does not have a table header, it should have one with scope="row"

  Does the layout look good on a range of screen sizes?

  • It looks a bit small on desktop.
  • The layout for the mobile is the same as the one on desktop.

  Is the code well-structured, readable, and reusable?

  • The HTML is readable.
  • The CSS could use a bit of structure with variables and better named classes.

  Does the solution differ considerably from the design?

  There are differences with the design such as colors, font weights and spacing, but the structure follows the design.

  00
• 

  Emmanuel•110

  @MrNaturi

  Posted 11 months ago

  You need to work on your styling and attention to details

  00