Design comparison
Solution retrospective
Hi! I just completed this challenge, any feedback is welcome! Thanks :)
Community feedback
- @Tryt4nPosted almost 2 years ago
In your
section-info
class would be better to use grid instead of divided it into twodiv
and use flexbox on them. You just could do something like this:display: grid
grid-template-columns: 1fr 1fr;
grid-template-rows: 1f 1ft;
And then it would be easier in mobile view:
grid-template-columns: 1fr;
grid-template-rows: repeat(4, 1fr);
Also you could add some transition while :hover elements. Now it's just so instant especially in your button. Give
transition: 300ms ease-in-out
and would bo smooth.About accessibility you done good job with creating a
label
although it has no content, but in that case you should give itaria-label: "Enter email"
0
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