Created using HTML5 , CSS, Flexbox, CSS Grid, mobile first approach

Community feedback

• 

  Miroslav•180

  @miroslavdurinPosted over 2 years ago

  Hello, you can try adding display: block in .mobile-imgs class. Something like this should work:

  .mobile-imgs {
    width: 100vw;
    
    display: block;
  }
  

  Marked as helpful

  0

  ---

  

  Nkiruka Ebele•90

  @nkirukkaPosted over 2 years ago

  @miroslavdurin it worked! Thank you so😁😁😁

  1
• 

  Abreham Nigussie•470

  @DreamCoder7Posted over 2 years ago

  Hey, Nkiruka Ebele👋

  Great effort on this challenge!👏

  A few things I'd like to suggest are,

  • Displaying the images from your html using img tag isn't problematic but you can take the advantage of background-image: url('..') to switch between the mobile version of your img to desktop version img. This way you can a void the repetition.
  • It's more interesting if you add a transition properties on the anchor tag <a></a> in your transform section
  • I also notice that the modal windows which is toggle or fire by clicking the hamburger menu it's not set to a high z-index value, In <section class="design-photography"></section> it overlap by the text. So you can quickly fix that and also adding a backdrop can help the modal to stand out from the page not relevant but also good practice.

  Hope you find this helpful!😊

  Happy coding😎

  0

  ---

  

  Nkiruka Ebele•90

  @nkirukkaPosted over 2 years ago

  @redstorm-hub thanks a lot for the feedback, will work on those

  0