Submitted almost 2 years ago

Mobile First Product Review Card Solution With CSS Flexbox and Grid

Aliyu-Saidu•220

@Aliyu-Saidu

Preview site View code

This is a solution for...Product preview card component

HTML
CSS

1newbie

Design comparison

SolutionDesign

Solution retrospective

Hi, I'm Aliyu and this is my solution for this challenge!

I added Semantic HTML5 markup, do you this this solution is accessible enough?
What is your opinion about the code, is it clean enough?
Where do you think I really need to improve on? Any link, tips or resource for general improvement will be appreciated.

Community feedback

suhayb jirde•1,050
@suhaybjirdePosted almost 2 years ago
Well done pro for my advice only add mobile version it doesn't look goo for the mobile🥰 Make it responsive
Marked as helpful
0
Aliyu-Saidu•220
@Aliyu-SaiduPosted almost 2 years ago
@suhaybjirde Oh thanks!

Any tips on how to fix that?
0
suhayb jirde•1,050
@suhaybjirdePosted almost 2 years ago
@Aliyu-Saidu use either flex or grid that way it easy to fix it Check out my solution as a reference
Marked as helpful
0
Aliyu-Saidu•220
@Aliyu-SaiduPosted almost 2 years ago
@suhaybjirde I have actually use both flex and Grid. Please check my style.css file. May be I did not do it right. But sure I will check your work. Once again thank you for your time.
0
suhayb jirde•1,050
@suhaybjirdePosted almost 2 years ago
@Aliyu-Saidu .wrapper { width: 350px; border-radius: 10px; margin: auto; } use then for your wrapper
Marked as helpful
0
Aliyu-Saidu•220
@Aliyu-SaiduPosted almost 2 years ago
@suhaybjirde Thank you so much. I will try that.
0

Please log in to post a comment

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