All comments

• 

  Mazen Sadek•140

  @CREPTONICESubmitted 5 days ago

  

  Fully Responsive Recipe page

  2

  20
  

  P

  schindlerdumagat•80

  @schindlerdumagatPosted 5 days ago

  Upon checking your html, I recommend that you should only h1 once. Put the "Simple Omelette Recipe" as the h1 and make "Preparation time", "Ingredients", "Instructions" and "Nutrition" as h2 because they are sub sections. It is a best practice to only use h1 once on every page and use h2 - h6 for the succeeding sub sections in order.

  Marked as helpful

  0
• 

  RatulSaha079•20

  @RatulSaha079Submitted 8 days ago

  

  social-links-profile-main

  1

  30
  

  P

  schindlerdumagat•80

  @schindlerdumagatPosted 7 days ago

  I can see that there are still inconsistencies with the styles. I assume this is not yet done.

  Marked as helpful

  0
• 

  Vincenzo Vitello•40

  @vincenzo-vitelloSubmitted 7 days ago

  

  Blog post card using css

  1

  30
  

  P

  schindlerdumagat•80

  @schindlerdumagatPosted 7 days ago

  I see that the positioning of the card is not in the center of the page but if you intend make it this way. That is ok.

  0
• 

  James Love•10

  @JamesTLSubmitted 26 days ago

  

  QR code component

  1

  30
  

  P

  schindlerdumagat•80

  @schindlerdumagatPosted 9 days ago

  The card is not positioned in the middle of the page. Upon inspecting your styles.css, you have used flexbox to move it to the center. There is just one thing missing, you haven't specified the total height of the body element which you specified as the flexible containing element by giving it a display of flex and it defaulted to the height of the card. Make sure to set the height of the body element to the total height of the browser window because flexbox also takes into consideration the height and width of the flexible containing element.

  You can do this by giving it a css rule of:

  height: 100vh;

  0