@CarlHumm
Posted
Hi there tonage, the solution looks good!
The preview looks just like the design and it's responsive aswell. My only criticism would be that you've set max-width: 85%
which is a relative percentage. On my monitor (2560x1440) the design appears very stretched because there is no real max-width. If you want it to take up 85% of the width for the most part, you could set the width to that and then set a maximum width and centre the grid. Like max-width: 1280px
or even use the clamp() function with the width property.
Other than that, good job. Interesting to see you using the attribute selector and matching to select all cards instead of using a unifying class or nth-selector. It's a good opportunity to learn for sure.
Good luck on your future challenges/work.
Marked as helpful
@tonage72
Posted
@CarlHumm thanks. Did just that. I only viewed it at 1920. That will be very helpful in the future.