-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Show confirmation dialog when deleting notes from the context menu #1749
Conversation
… set in preferences
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution!
please confirm my review!
browser/main/NoteList/index.js
Outdated
@@ -574,22 +574,29 @@ class NoteList extends React.Component { | |||
}) | |||
} | |||
|
|||
confirmDeleteNote (selectedNotes) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This function is duplicated of it.
I want you to extract to browser/lib
.
Sorry for the delay things have been super busy irl. @sosukesuzuki Thank you for making those changes 👏 |
Fix for #1726
Delete confirmation dialog will show when deleting a note from the context menu when
Show a confirmation dialog when deleting notes
is checked in preferences.