taskFIX #1481
taskFIX #1481
Conversation
IOherhi
commented
Mar 5, 2025
IOherhi
commented
There was a problem hiding this comment.
Thank you for your effort on this task! 😊 However, there are a few critical issues that need to be addressed before we can approve the pull request:
-
Syntax Error: There's a syntax error on line 6 due to the incorrect declaration of the variable
fielD. This prevents the script from running. Please correct it toconst field = body.querySelectorAll('.field');. -
Missing
<label>Elements: The task requires creating and appending<label>elements for each input with the classfield-labeland theforattribute set to theidof the current input. This logic is missing from your code. -
Placeholder Capitalization: While you correctly set the
placeholderattribute, the capitalization is not as specified. Instead of usingtoUpperCase(), you should capitalize only the first letter of the name. -
Appending Labels: The created
<label>elements should be appended to the parent container of each input, which is currently not implemented.
Please address these issues to meet the core requirements of the task. Once these changes are made, your solution will be much closer to completion. Keep up the good work, and don't hesitate to reach out if you need further assistance! 🚀
✨ Please rate this review in the Platform's notification center. I will be grateful for your feedback! ✨
🔄 Fix the issues, commit the changes, and then re-request my review using the Re-request review button in the Reviewers tab.
| const body = document.body; | ||
| const allInputs = body.querySelectorAll('input') | ||
| const arrayInputs = [...allInputs]; | ||
| const fielD body.querySelectorAll('.field'); |
There was a problem hiding this comment.
There is a syntax error here. The variable fielD is not correctly declared. It seems like you intended to declare a constant for elements with the class 'field'. You should correct this to const field = body.querySelectorAll('.field');.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
There was a problem hiding this comment.
Sorry, the pull request review was unsuccessful for the following reasons:
Warning
No changes were made in the pull request since the last review. Please make sure that you have added files to the pull request and committed the changes.
[CODE: 7]
If the issue is within your control, please address it.
The next review attempt will automatically launch in 1 hour.
If you believe this is an error or need assistance, please contact the support team.
| const body = document.body; | ||
| const allInputs = body.querySelectorAll('input') | ||
| const arrayInputs = [...allInputs]; | ||
| const fielD body.querySelectorAll('.field'); |
