Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Recipe Page

@Dnyanesh-Bachhav

Desktop design screenshot for the Recipe page coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Community feedback

P

@iddahadev

Posted

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.

0
Emmanuel 70

@MrNaturi

Posted

You need to work on your styling and attention to details

0

Please log in to post a comment

Log in with GitHub
Discord logo

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord