Responsive landing page using CSS Grid and FlexBox

Community feedback

• 

  Suzunatsu•130

  @st0272

  Posted 10 months ago

  Your code is well-structured, but there are several important points for improvement:

  • The desktop design is slightly smaller compared to the original design. Please make sure to measure the padding, image sizes, and other elements correctly.
  • The mobile version of the design is the same as the desktop version. Please create a mobile version using appropriate media queries.
  • Ensure that the h4 tag with the .header__caption class follows the appropriate heading structure. It's better not to use an h4 tag before an h1. It might be better to use a p tag instead of an h4.
  • The h2 and h3 tags inside the section tag with .main__pricetag class might confuse the document hierarchy. You could consider using <span> with appropriate class names and using <div> tag or <p> tag instead of <section> tag.

  I think your coding skills could be even better. and Keep up the good work!

  00
• 

  Danil-Shevchenko•150

  @Danil-Shevchenko

  Posted 10 months ago

  Great job

  00