vanilla multi page form

Solution retrospective
Any feedback?
Please log in to post a comment
Log in with GitHubCommunity feedback
- @MaciejNarejko
Hello there 👋
Here's the feedback:
-
When the user goes to at least the second step and then goes back to the first step, the "Go Back" button should be not available in the first step. Add a condition in the goBack() function to handle the case when currentPageNumber is 1.
-
After entering all fields incorrectly on the first page of the form, and then correctly filling in some of them, the error message for those correct fields does not disappear.
-
The background and foreground colors do not provide sufficient contrast ratio, which can strain the user's eyes. Consider adjusting the color scheme to ensure better readability and accessibility.
-
The document lacks a meta description tag <meta name="description">.
-
Remove unused code, such as the addOnLabels array: const addOnLabels = ["Online Service", "Larger Storage", "Customizable Profile"];
-
Replace occurrences of var with let or const for consistency. For example: let label = checkbox.parentNode;
-
Consider extracting reusable code into helper functions to improve modularity and readability.
I hope I provided at least some useful feedback. Happy coding 🙂
Marked as helpful -
Join 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