Submitted almost 3 years ago
Using flexbox and grid layout
@RafaelAF
Design comparison
SolutionDesign
Solution retrospective
open to sugestions
Community feedback
- @darryncodesPosted almost 3 years ago
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-container
as 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 helpful0 - i'd remove
Please log in to post a comment
Log in with GitHubJoin our Discord community
Join thousands of Frontend Mentor community members taking the challenges, sharing resources, helping each other, and chatting about all things front-end!
Join our Discord