Design comparison
Solution retrospective
Brushing back up on code after a long break after leaving code bootcamp/tech school. First project on this platform. Still need to make a few changes in the media queries folder. Feel free to critique.
Community feedback
- @pikapikamartPosted over 3 years ago
Hey, great work on your first challenge here in FEM!
Though I need to zoom out because I can't see the desktop layout. Are you working on a large screen?
Okay so the desktop layout is pretty good, but going in mobile state, resizing your layout is destroyed. The contents are overlapping one another and hiding things.
This was supposed to be a grid challenge right, and using grid on this one is really beneficial for you so that you can learn about it. So instead of using
display: flex
and usingposition: relative
on those cards. You could make use of grid and it will make it more pretty and more easy to handle.Because that layout is designed for grids right, though it can be done in flex but use what will make it easier. You can check my solution on this one and I can help you if you need anything about using grids okay.
So for now, good work on the desktop layout and I really appreciate if your refactor it okay^^
0@erickwalker1Posted over 3 years ago@pikamart Yes, I'm working from a 15-inch MB Pro. And okay, thanks for the feedback, I've heard of grid but have never been introduced to it so that will be something to pick up.
0
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