Skip to content
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

Allow binding mouse buttons to commands #2572

Closed
riverbl opened this issue Aug 27, 2023 · 5 comments · Fixed by #2573
Closed

Allow binding mouse buttons to commands #2572

riverbl opened this issue Aug 27, 2023 · 5 comments · Fixed by #2573
Labels
C-feature Category: New feature or request

Comments

@riverbl
Copy link
Contributor

riverbl commented Aug 27, 2023

I'd like to bind the side buttons on my mouse to navigate forward and backward. This behaviour is the default in web browsers and VS Code.

Currently Lapce doesn't allow binding mouse buttons to commands, so this is not possible.

@riverbl riverbl added the C-feature Category: New feature or request label Aug 27, 2023
@panekj
Copy link
Collaborator

panekj commented Aug 27, 2023

It's already implemented

@riverbl
Copy link
Contributor Author

riverbl commented Aug 27, 2023

Ah OK - how do I bind mousebackward and mouseforward to navigate backward and forward? The following does not work for me on master:

[[keymaps]]
command = "jump_location_forward"
mode = "i"
key = "mouseforward"

[[keymaps]]
command = "jump_location_backward"
mode = "i"
key = "mousebackward"

@panekj
Copy link
Collaborator

panekj commented Aug 27, 2023

It's implemented in latest release, if it doesn't work while building from source, it should be mentioned in #2417

@riverbl
Copy link
Contributor Author

riverbl commented Aug 27, 2023

I don't see it mentioned in #2417. I'm building the latest master from source.

@MinusGix
Copy link
Member

(The idea is/was that features that were implemented in release but were missing from master would be added to #2417 to make it easier to track that we need to reimplement them)
I added it and also linked your PR. Thanks for reimplementing it, I was missing that feature.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-feature Category: New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants