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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggestions for improvement #7

Open
4 tasks
levy002 opened this issue Feb 24, 2022 · 0 comments
Open
4 tasks

Suggestions for improvement #7

levy002 opened this issue Feb 24, 2022 · 0 comments

Comments

@levy002
Copy link

levy002 commented Feb 24, 2022

Hi @Nemwel-Boniface 馃憢 ,
Congratulations on your project, you put in great effort to work on this project and that is highly commendable.
However, there are a few suggestions I will like you to implement;

  • It should be better after every change happens in local storage, we must make sure that the task.index is updated
  • It should e better to validate the input.value before adding the description to the list, to make sure that no empty value or duplicated value is being added to the list
  • It could be better to make your web app more secure by using only secure plugins/CDN, you can do that just by HTTPS instead of HTTP.
  • list overflow after adding a large number of tasks: link.

Happy coding!!

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