reciepe page

Community feedback

• 

  paulDev•660

  @dev-paulLPosted 4 months ago

  Hi, quick review:

  • <main> landmark is present, that's good
  • missing headings (you used h1, but no h2 nor h3 for the rest : ingredients, instructions, nutrition are headings for example)
  • <strong> instead of <span> => good. But some of your <span> are useless
  • correct use of <ol> and <ul>
  • incorrect image path
  • incorrect image alt
  • using pixels for font-size is a bad habit for accessibility, instead use 'rem' (1rem = 16px), they adapt to the browser font size.
  • width: 100%; max-width: 80%; on your .wrapper?
  • repeated css properties (margin & padding).
  • missing some styling

  happy coding 🫡

  0