html, css, flexbox, js, mobile-first, media-queries

Community feedback

• Account deleted

  On my screen the 'immersive experience...' is on top of the navigation bar, and the photos have no spacing between them.

  Marked as helpful

  1
• 

  Mojtaba Mosavi•3,760

  @MojtabaMosaviPosted over 3 years ago

  Hi, your solution looks good and responds well, The quite a few small sublte things that you can take your solution to next level.

  1. Adjust the font size of hero title on desktop view.

  2. Whenever there is a specific tag for a certain taks use it because it makes your markup more semantic, for example about-img you should use an image tag instead of div.

  3. For you media queries try to use relative units either rem or em, I recomand you use ems because it improves the responsibility of you solution across different browsers.

  4. The icons in the footer are missing the underline.

  Keep coding :)

  Marked as helpful

  0
• Account deleted

  You can Also check the report for some html issues.

  0
• Account deleted

  That's Great. I like it 😍.. You can do a transition for photos when hover.. and check the font-size .

  0