@yishak621
Posted
hi ken congrats for completing the challenge two basic things i want to comment about are
1]the scale of the card is too big it fits the ahole screen so you should decrease it like these ...go to the main container for the card and transform:scale(0.7)
and also u didn't design for small screens (mobile version)...the main challenge for this specific project is to design responsive card for pc version(1row with 2column grid) and for mobile(1column with 2rows) and the images for each version is different in the images folder u will find two images for two source sets ...these practical challenge helps me to practice how to set a resolution switching image according to the screen sizes so we have given two diffrent images 1x and 2x for small and wider screens so whenever the screen size changes from 750px the image we provide will be changed automatically! these is how it is done
<picture> <source srcset="images/[email protected] 750w, images/[email protected] 1200w" /> <img src="images/[email protected]" /> </picture>
beside the card grid layout will be change from 1x2 to 2x1 for mobile versions !
and also it have something to practice on text-decoration property for price tag
we must put line-through and the thickness should be very thin `<div class="old__price">
.old__price__tag { font-weight: 400; text-decoration-line: line-through; text-decoration-thickness: 0.1em; }
Marked as helpful
@askan-ken
Posted
@yishak621 Thank you for your help.