Simple Omelette Recipe page

Community feedback

• 

  Pritam Panda•60

  @imxeonPosted 5 months ago

  You have achived the design with slightly different code, great work 👏 but here are some minor improvements that you can make from my perspective.

  0. The design of the card in desktop view is quite broad which does not looks familiar to the reference design quickly fix it.
  1. You can use section element with an id to name it, to wrap related div elements.
  2. Use less div elements.
  3. The fifth div element is using the wrong class name rectify it for acessibility.
  4. Write the full name for sixth div class - nutrition instead of nutri
  5. Remove the margin from the main element for the mobile view.
  6. The body type selector is repeated in the stylesheet twice. Move the property values from the second body type selector to the first one or vice versa.
  7. Your design is lacking the spacing after the unordered lists and ordered lists, Add the padding-left property to add the space.

  .prep-time ul li, .ingredients ul li, .ordered li {
      margin-top: 0.7rem;
      color: hsl(30, 10%, 34%);
      padding-left: 20px;
  }
  

  0