Skip to content
Submitted almost 4 years ago

Flexbox and very small amount of Java and Grid

Igor•45
@IgorVanVuuren
A solution to the Agency landing page challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Was a fun challenge learned a lot of new stuff.

If there is anything anyone can see that i should work on plz tell me :)

Code

Please log in to post a comment

Log in

Community feedback

  • Chamu•13,860
    @ChamuMutezva
    • look into responsive images techniques mainly picture element or srcset, those are considered part of responsive web design. The method you used below works but is not part of responsive web design - it is considered among other things expensive and uses a lot of bandwidth as all images have to be downloaded.
    <div class="nav-wrapper">
 <img class="mobile-hero-images" src="images/mobile/image-header.jpg" alt="">
 <img class="desktop-hero-images" src="images/desktop/image-header.jpg"  alt="">
  </div>

    ***** &&&&&&&&&&&&&&&&&& ***

    <img class="hamburger" src="images/icon-hamburger.svg" alt="">
<img class="cross hide" src="images/icon-close.svg" alt="">
this should have been  interactive elements -  button
    • semantic matters , use elements that are meant for the task.
    Marked as helpful

Join our Discord community

Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!

Join our Discord