You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This surprisingly (from a user perspective) does not actually disable the right mouse command, because the merge uses the default value which is not nil.
A workaround is to set the command to '' instead of nil.
I'm not sure there is any solution, maybe just a note in the documentation that putting a nil in the config will have no effect and to use '' instead in this situation.
Yeah in the case of user configurations setting things to nil is unfortunately not a good way to indicate that it isn't wanted for the reason specified above. You can also use false as a value to prevent the command being run. I've added a note to the docs on the dev branch
Is there an existing issue for this?
What happened?
I wanted to disable the right mouse click action.
Because the user-provided config is merged with the defaults, setting an option to
nil
has no effect. For exampleThis surprisingly (from a user perspective) does not actually disable the right mouse command, because the merge uses the default value which is not
nil
.A workaround is to set the command to
''
instead ofnil
.I'm not sure there is any solution, maybe just a note in the documentation that putting a
nil
in the config will have no effect and to use''
instead in this situation.What did you expect to happen?
Since I saw
in the default options, I thought I could simply swap the values to switch the commands. But I had to replace
nil
with''
as explained above.Config
Additional Information
No response
commit
No response
The text was updated successfully, but these errors were encountered: