Implement turning off y/n prompt for file deletion #1887
Conversation
There was a problem hiding this comment.
We should try to avoid adding new options if at all possible. If the is highly desirable I would like that option name to be changed to something like:
ui = {
confirm = {
node_deletion = true,
}
}(node can actually be called file, because everything is a file :)
Would you be willing to add other confirmations as well?
Yes. Some of these are annoying, but it would be good to be selective. How about we add an enable to turn them all off? ui = {
confirm = {
enable = true,
node_deletion = true,
paste = true,
trash = true,
create = true, |
lua/nvim-tree.lua
Outdated
| @@ -495,6 +495,7 @@ local function setup_autocommands(opts) | |||
| end | |||
|
|
|||
| local DEFAULT_OPTS = { -- BEGIN_DEFAULT_OPTS | |||
| file_deletion_confirmation = true, | |||
There was a problem hiding this comment.
Please refactor this to ui.confirm.remove.
|
Made refactors as suggested |
There was a problem hiding this comment.
Remove works beautifully, trash fails at cf36d35 with:
E5108: Error executing lua: ...cker/start/amamdemous/lua/nvim-tree/actions/fs/trash.lua:32: attempt to index field 'trash' (a nil value)
stack traceback:
...cker/start/amamdemous/lua/nvim-tree/actions/fs/trash.lua:32: in function 'handle_tree_actions'
...cker/start/amamdemous/lua/nvim-tree/actions/dispatch.lua:123: in function 'dispatch'
...k/packer/start/amamdemous/lua/nvim-tree/actions/init.lua:273: in function <...k/packer/start/amamdemous/lua/nvim-tree/actions/init.lua:272>
| remove = true, | ||
| trash = true, | ||
| }, | ||
| }, |
There was a problem hiding this comment.
Next time please remember to update help as per CONTRIBUTING
Added doc for this section and renamed to match the action.
…uire_confirm (#1887) * fix(#1878): nvim frozen on no name buffer when modified.enable (#1879) * feat(api): add api.config.mappings.active, api.config.mappings.default (#1876) * feat(api): add config.mappings.current and config.mappings.default * feat(api): add config.mappings.current and config.mappings.default * feat(api): add config.mappings.current and config.mappings.default * Implement turning off y/n prompt for file deletion * Refactor different prompt configs to single ui table * Remove unused fields * add ui.confirm doc, format/tidy * trash.require_confirm -> ui.confirm.trash * Fix nil value trash field Co-authored-by: Richard Li <38484873+chomosuke@users.noreply.github.com> Co-authored-by: Telman Babayev <babayevtolman@gmail.com>
I have a suggestion to give users the option to disable the prompt when deleting a file. For example, when working in small folders, there is no particular need to double-check the file you are deleting