Skip to content
Submitted almost 4 years ago

Responsive page with CSS Grid, Flexbox and a little bit of JavaScript

Alessandra do Couto•25
@alessandradocouto
A solution to the Agency landing page challenge
Preview (opens in new tab)Code (opens in new tab)

Solution retrospective

Hey, guys!! I did my best to implement the features and I'm grateful if you could get ideas on how I can improve. Thank you very much.

Code

Please log in to post a comment

Log in

Community feedback

  • Brooke Wargnier•155
    @brookewargnier

    I'd focus on the mobile experience next! You have the webpage set up and the images move to separate lines, but the nav bar and "we are creatives" element completely disappears.

    Looking at your code, it looks like your class "h-container" is not included in your media query beyond 758px.

    Another thing you could look into that was suggested to me by another member is BEM methodology for naming your elements and organizing your CSS. This video gives a great overview: https://www.youtube.com/watch?v=er1JEDuPbZQ&t=782s

    This really helped my organization of projects and made it so everything was more legible and easy to locate in my CSS files.

    Marked as helpful
  • Fernando FeN•425
    @Fernando0654

    Hi! For some reason the Design and Photography section text shows me the entire div at the top of the images. I'd like to share screenshot, but I can't.

    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