Design comparison
Solution retrospective
any feedback is welcome!!
Community feedback
- @FluffyKasPosted about 3 years ago
Hey, your solution looks great, responsiveness seems fine too! The only thing I noticed, you forgot to include the email validation in your function so now it only throws an error if you try to submit your email empty ^^
1@superozzyPosted about 3 years ago@FluffyKas thank you 😃i will check it out and upadate my solution!
0 - @Dharmik48Posted about 3 years ago
Hey👋,
Great job with the solution😄! It looks really nice, just a few issues:
- Your hierarchy is like
body
->.container
->wrapper
, and here I don't think that the.container
or.wrapper
, any one of these is unnecessary. - Also try to add
transition
to thehover
states.
Apart from these, it is really good, Keep Developing👍.
0@superozzyPosted about 3 years ago@Dharmik48 hey thanks for the tips,i keep both of the classes because it seemed easier to me when the time came for the media queries! I will also check the transition!Thank you for your time!
0 - Your hierarchy is like
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