darryncodes• 6,430
@darryncodes
Posted
Hi Antonio,
Really good solution and very responsive.
With a few tweaks i think it could be perfect:
- i'd remove
height: 70%;to.card-containeras it's creating a really long card on desktop, you'll need space things out inside the card a little more to match the design - it might be cool to look into button styling to help match the desired hover effects to the design
- you could clear your accssibility/html report if you mad this
<section class="container">a<main class="container">and this<div class="attribution">a<footer class="attribution">. semantic html is really important and fundamental
Hope that helps
Marked as helpful
0
