@tesla-ambassador
Posted
Hey Troy! It's great to see that you have uploaded yet another awesome solution.
Your report is looking so clean, kudos! π
Anyways It's responsiveness is consistent on all screens but I think it'd be a lot better if you switched to one grid column at a minimum screen width of 650px
because your card component still fits in the layout even at that screen width. (I also recommend using pixels instead of em
or rem
to specify screen widths in your media queries)
- You might also want to add a hover state to your button to give it a more interactive feel and make it look less static. Otherwise, this is a very solid solution, keep up the great work and keep on coding! π
Marked as helpful
@troy03
Posted
@tesla-ambassador i watch a tutorial explaining why use of em in media queries hahaha thanks up to now I'm studying sass and creating a project using css precessor.
@tesla-ambassador
Posted
@troy03 Great work Troy, keep it up! I also enjoy using sass, it makes writing css a lot more easier.