-
Notifications
You must be signed in to change notification settings - Fork 294
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
need to fix - onChange Event Issues #77
Comments
It looks like this is your first issue. Welcome! 👋 |
attempt |
I was able to reproduce the issue on https://mdn.github.io/todo-react/: Screen.Recording.2023-10-02.at.13.23.54.mov |
The reason is todo-react/src/components/Todo.js Lines 41 to 48 in 5dc43fa
As soon as Instead: However, it is then important to ensure that the todo item cannot be saved with an empty name (just like "Add" doesn't do anything if the field is empty). |
value={isEditing ? newName : props.name} Just do these changes and it will work |
|
Thanks all, I'm closing this one as done now! Well done on the great approaches to improving the example! 👏🏻 See |
What information was incorrect, unhelpful, or incomplete?
How I am facing issues => When I click on Edit on any Todo and try to clear the entire field, I get the original text back again.
The issue occurred in => the Edit-Template.
What did you expect to see?
The Input Field Must be completely Clear and User can type Entirely New Name of Todo.
Do you have any supporting links, references, or citations?
No response
Do you have anything more you want to share?
That is enough to get a reference to the actual error point.
The text was updated successfully, but these errors were encountered: