Design comparison
SolutionDesign
Solution retrospective
Any suggestions are welcome on this solution
Community feedback
- @0xabdulkhaliqPosted over 1 year ago
Hello there 👋. Congratulations on successfully completing the challenge! 🎉
- I have other recommendations regarding your code that I believe will be of great interest to you.
SUGGESTIONS 🟡:
- The success state is not seen because of the page reloads before the success element is displayed with flexbox
- This is due to form submission, you want to add
e.preventDefault()
method to form submission to prevent the default submitting behaviour
- And along with that you can consider to wrap the
section
element with class ofsuccess
inside themain
element
- Because
section
element can't define the landmark for a site, it can only be used within landmark elements likemain
- So make sure to nest the
section
insidemain
to improve accessibility
.
I hope you find this helpful 😄 Above all, the solution you submitted is great !
Happy coding!
Marked as helpful0@devjhexPosted over 1 year ago@0xAbdulKhalid Man thanks for the help it was really helpful.
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