Overall, really nice work. I like your use of media queries to incorporate responsiveness in your design. The use of CSS variables is a nice practice too.
Some minor suggestions to consider:
- the content container's padding isn't aligned to the design, which is causing your h1 heading to not look like the designs.
- put all your right-side content into one container so that one padding property will align everything. Notice that your 'perfume' text is a bit left of the heading, and your button element is to the right of the pricing text. Ideally, this should have the same vertical left-side alignment.
- old price text should be smaller. consider the <small> text.
Hope these help! Again, nice job!
Marked as helpful
@TheBeyonder616
Posted
@maxkdavis thanks you I’ll keep that in mind