-
Notifications
You must be signed in to change notification settings - Fork 41
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
Mouse scroll support and select multiple commands for further manipulation. #200
Comments
…lity to highlight text This reverts commit 7d9bb66.
Thanks for filing this! I've spent a bit of time on this, and interestingly, even just supporting scrolling via the mouse is actually a bit difficult to support. Adding it was trivial (see 7d9bb66), but it turns out that this breaks the ability to highlight text in order to copy/paste it. Personally, I find myself copy/pasting text from the TUI decently often (e.g. if I want to copy/paste just a directory name) so I'd rather not break this. I'm still looking into the feasibility of this, but it is tricky since the underlying library we're using for terminal interactions doesn't support capturing the scroll wheel without capturing all other mouse events. I still plan on spending some more time looking into this, but this might have to be supported behind an opt-in config option. |
The text was updated successfully, but these errors were encountered: