-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Solution #1476
Solution #1476
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job on implementing the task requirements! 🎉 Your code effectively creates labels for each input and assigns IDs where needed. While the placeholders match the labels, consider if they should provide additional guidance. Also, ensure input names are unique and user-friendly to avoid any potential issues. Keep up the good work! 😊
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
if(!input.id) { | ||
input.id = input.name; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This block assigns an ID to inputs that do not have one, using the input's name. Ensure that input names are unique to avoid duplicate IDs, which can cause issues in the DOM.
} | ||
|
||
label.setAttribute('for', input.id); | ||
label.textContent = input.name.charAt(0).toUpperCase() + input.name.slice(1); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The label's text content is set by capitalizing the first letter of the input's name. Ensure that input names are descriptive and user-friendly, as they will be displayed as labels.
|
||
label.setAttribute('for', input.id); | ||
label.textContent = input.name.charAt(0).toUpperCase() + input.name.slice(1); | ||
input.placeholder = label.textContent; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The placeholder is set to the same value as the label's text content. Consider whether this is the desired behavior, as placeholders typically provide additional guidance or examples for the user.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure to deploy your page and provide a link to the demo in the PR description
No description provided.