Responsive page with CSS Grid, Flexbox and a little bit of JavaScript
Design comparison
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.
Community feedback
- @brookewargnierPosted about 3 years ago
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 helpful2@alessandradocoutoPosted about 3 years ago@brookewargnier Brooke, this video is so awesome! I reviewed the code and could understand where the bug was. Thanks for taking the time to help me, I'm so grateful for your help!
0 - @Fernando0654Posted about 3 years ago
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 helpful1@alessandradocoutoPosted about 3 years ago@Fernando0654 I fixed this error that I hadn't noticed before. Thank you so much from the bottom of my heart, Fernando!
1
Please log in to post a comment
Log in with GitHubJoin 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