responsive product preview card using sass

Please log in to post a comment
Log in with GitHubCommunity feedback
- P@lenny131
Good job on your solution. It looks like the design, and the layout changes correctly with the width of the browser window.
Something you might want to be careful about is the way you use both rem and px for sizing. The page looks correct when the browser's default font size is 16px, and the zoom function works fine as well. However, when changing font size in the browser settings, some of the text resizes, like the header and price, but the description stays at 14px.
One other minor suggestion is to consider moving the button's
transition
property directly under thebutton
selector, rather than underbutton:focus, button:active, button:hover
. As it is right now, the color transition only works when entering the hover (and other) states, but, when exiting, the color changes instantly instead of using the transition.Marked as helpful
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