Closed Mefisto04 closed 9 months ago
Name | Link |
---|---|
Latest commit | c5ab7a2a481c37a68f3ad4d365c4653d5e711e28 |
Latest deploy log | https://app.netlify.com/sites/frontend-database/deploys/652189d0963fb50008f1ce29 |
Deploy Preview | [https://deploy-preview-24--frontend-database.netlify.app/projects/typing progress bar](https://deploy-preview-24--frontend-database.netlify.app/projects/typing progress bar) |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
Hey there! I've created a report for your pull request. Let's dive in!
ProjectList.json
file.index.html
for the "Typing Progress Bar" project.script.js
for the "Typing Progress Bar" project.styles.css
for the "Typing Progress Bar" project.script.js
file. For example, instead of wordDisplay
, you could use typingDisplay
to make it clearer.script.js
file.No bugs found.
styles.css
file to an external file or use inline styles for better separation of concerns.In the script.js
file, the following code snippet can be refactored for better readability:
wordInput.addEventListener('input', function () {
const inputText = wordInput.value;
const expectedWord = wordsArray[currentWordIndex];
if (inputText === expectedWord) {
currentWordIndex++;
wordInput.value = '';
updateProgressBar();
if (currentWordIndex === wordsArray.length) {
alert('Congratulations! You typed all the words correctly.');
} else {
wordDisplay.textContent = wordsArray[currentWordIndex];
}
}
});
I would rate the code a 7 out of 10 based on the following criteria:
That's it for the report! Let me know if you need any further assistance. Keep up the good work! 👍
Describe your change:
Checklist:
ATTACH SCREEN-SHOTS