Responsive social proof section master with hover and select effect
Design comparison
Community feedback
- @dylan-dot-cPosted 5 days ago
Hey I think I know whats the issue here!
I think its the media query you have
max-width: 1350px
is too large as this means for 0 - 1350px screens would have the mobile design. I think you should use a lower breakpoint value like 1024px. And make sure your text at the ratings don't wrap, you can disable that feature withtext-wrap
in css.Also try not to use absolute with and height in css, try to make it more natural by letting grid/flex decide that.
Also I see you using margin to add spacing away from the corners, according to the css box model, you should use padding to space from the corner, use padding on the
.card
class and not margin on the.comment
class.You should also make the avatar component a div and use flexbox to align it properly not position absolute.
I suggest you look up the box model, css flexbox and grid to better understand the best ways to build layouts for your webpages
0@YussefMoPosted 4 days ago@dylan-dot-c thanks that was an old project for me i forgot to upload it and i start using the flex align you can check it in my blog card challenge instead of position
0@dylan-dot-cPosted 4 days ago@YussefMo so what u can do is update this solution. It's good to go back to previous work and improve them Gives you a sense of improvement from when you were younger/beginninger
0@YussefMoPosted 1 day ago@dylan-dot-c i've update the Media Query for this challenge
0 - @YussefMoPosted 5 days ago
IDK why the screenshot like this but it display will like the design in 1440px width screen and 1400px
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