Aryarishi Sikdar• 270
@ArryBlack
Posted
Layout looks great.
- One issue I notice is that the card starts to overflow on smaller screens (starting around 396px and below). Instead of setting the width of the card a fixed rem value, you should opt for a percentage value, setting in max and min-width values based on your preference. That is the only thing you need to change to make this layout responsive, no media queries necessary. For reference, I set a width of 30% for my card. I set min-width:256px (80% of 320px), which is the smallest screen size to optimize for. You can set the max-width based on your preference.
- The fonts on your buttons don't match with the given design, although they look good.
- You can make your markup more semantic, maybe add the card into a main tag.
Btw, I solved the same challenge recently, any feedback on it will be appreciated, thank you!
Marked as helpful
1
LorenzoSerra1• 50
@LorenzoSerra1
Posted
You're right, thanks for the tip, it helped. I really appreciated mate @ArryBlack
1
Aryarishi Sikdar• 270
@ArryBlack
Posted
@LorenzoSerra1 You're welcome!
1
