Skip to content

Bug fix: The key mappings that are <silent> prevent Vim from asking the user to enter the encryption key - #235

Merged
justinmk merged 1 commit into
justinmk:masterfrom
jamescherti:remove-silent-key-mappings
Dec 8, 2022
Merged

Bug fix: The key mappings that are <silent> prevent Vim from asking the user to enter the encryption key#235
justinmk merged 1 commit into
justinmk:masterfrom
jamescherti:remove-silent-key-mappings

Conversation

@jamescherti

@jamescherti jamescherti commented Dec 8, 2022

Copy link
Copy Markdown
Contributor

Problem: The Dirvish key mappings that are <silent> prevent Vim from asking the user to enter the encryption key when an encrypted file is opened from Dirvish (:help encryption).

Solution: Remove <silent> from the key mappings that call dirvish#open() and replace :call with <cmd>call.

… '<cmd>call'.

Problem:
Dirvish key mappings that are <silent> prevent
Vim from asking the user to enter an encryption key when an encrypted
file is opened from Dirvish (`:help encryption`).

Solution:
Remove <silent> from the key mappings that call
'dirvish#open()' and replace ':call' with '<cmd>call'.
@jamescherti jamescherti changed the title dirvish#open() key mappings: Remove <silent> and replace ':call' with '<cmd>call' Bug fix: 'dirvish#open()' key mappings prevent Vim from asking the user to enter an encryption key Dec 8, 2022
@jamescherti jamescherti changed the title Bug fix: 'dirvish#open()' key mappings prevent Vim from asking the user to enter an encryption key Bug fix: The key mappings that are <silent> prevent Vim from asking the user to enter an encryption key Dec 8, 2022
@justinmk
justinmk merged commit 6233243 into justinmk:master Dec 8, 2022
@justinmk

justinmk commented Dec 8, 2022

Copy link
Copy Markdown
Owner

Thanks!

@jamescherti jamescherti changed the title Bug fix: The key mappings that are <silent> prevent Vim from asking the user to enter an encryption key Bug fix: The key mappings that are <silent> prevent Vim from asking the user to enter the encryption key Dec 9, 2022
monadnocking added a commit to monadnocking/vim-dirvish that referenced this pull request Mar 29, 2024
Problem:
`{Visual}O` and similar commands only operate on a single file
after 6233243 (fix(mappings): prefer <cmd>, drop <silent> justinmk#235,
2022-12-08).

Solution:
Use `line('v')` and `line('.')` to get the visual range when
the `<Cmd>` mappings are used.
monadnocking added a commit to monadnocking/vim-dirvish that referenced this pull request Mar 29, 2024
… file

Problem:
`{Visual}O` and similar commands only operate on a single file
after 6233243 (fix(mappings): prefer <cmd>, drop <silent> justinmk#235,
2022-12-08).

Solution:
Use `line('v')` and `line('.')` to get the visual range when
the `<Cmd>` mappings are used.
justinmk pushed a commit that referenced this pull request Apr 3, 2024
Problem:
`{Visual}O` and similar commands only operate on a single file
since 6233243 #235.

Solution:
Use `line('v')` and `line('.')` to get the visual range when
the `<Cmd>` mappings are used.
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

Successfully merging this pull request may close these issues.

2 participants