Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

Submitted

Perfume Presentation Panel - Flex Grid responsive

P
Sam Hooker 410

@35degrees

Desktop design screenshot for the Product preview card component coding challenge

This is a solution for...

  • HTML
  • CSS
1newbie
View challenge

Design comparison


SolutionDesign

Solution retrospective


What are you most proud of, and what would you do differently next time?

Loved using Grid combined with responsiveness

What challenges did you encounter, and how did you overcome them?

spacing and general grid sizing in responsive mode

Community feedback

Edu209C 80

@Edu209C

Posted

You should change the color of the title and the background color

0

@Lescano713

Posted

Variable Naming: Improve variable names for better clarity. For instance, --stone-100 and --stone-150 could be renamed to reflect their roles more clearly, such as --light-cream and --medium-cream.

Redundant Properties: Remove redundant properties. For instance, box-sizing: border-box; is already applied universally to all elements with * { box-sizing: border-box; }, so you don’t need to reapply it elsewhere.

0

Please log in to post a comment

Log in with GitHub
Discord logo

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