Skip to content
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

Check the UI bugs mentioned below once Pages has updated #20

Open
DFLLinton opened this issue Nov 15, 2022 · 0 comments
Open

Check the UI bugs mentioned below once Pages has updated #20

DFLLinton opened this issue Nov 15, 2022 · 0 comments

Comments

@DFLLinton
Copy link
Collaborator

There are quite a few errors that show up in the console that mention a Reference error or are related to the testing JavaScript file.

Only one test is running (Should add taskinput to list). The delete test is not running and does not display any output onto the console.

Array and Input box does not appear empty when you start up the app. Consider using taskInput.innerHTML= ""; to empty the input box.

When the array and input box test are commented out (see image below)
image

The error message is output multiple times upon reloading the page. This maybe because of the click event being run without an input in multiple tests.

Consider using LET and CONST instead of VAR. VAR is a global variable and can often cause errors and can sometimes be hard to pinpoint where the error is coming from.

It is best practice to define functions outside of another function that calls it. Consider moving the enterTask function outside of the raiseError function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant