Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add mouse support for the sidebar and output pane - #25

Closed
cyrus-fa wants to merge 5 commits into
laravel:mainfrom
cyrus-fa:feat/mouse-interaction
Closed

Add mouse support for the sidebar and output pane#25
cyrus-fa wants to merge 5 commits into
laravel:mainfrom
cyrus-fa:feat/mouse-interaction

Conversation

@cyrus-fa

@cyrus-fa cyrus-fa commented Sep 9, 2026

Copy link
Copy Markdown

Right now, navigating between processes and scrolling through output only works with the keyboard. Scrolling can be a bit slow, especially when there is a lot of output.

This PR adds mouse support for selecting processes in the sidebar and scrolling through the output pane with the mouse wheel.

@taylorotwell
taylorotwell marked this pull request as draft September 9, 2026 14:20
@joetannenbaum

Copy link
Copy Markdown
Collaborator

Thanks for this PR! I have some concerns here:

  • Scrolling output with the mouse should already work, as long as the pane is focused (I understand that's part of the intention of the PR, to scroll where the mouse is positionally, but just stating the size of the win here)
  • In testing this, I seem to have lost the ability to select text normally. Seems like I can get it back by holding down the option key but that doesn't feel very intuitive.
  • A good chunk of this work is re-inventing Ink's createInputParser, not clear as to why

@cyrus-fa

cyrus-fa commented Sep 9, 2026

Copy link
Copy Markdown
Author
  • Scrolling output with the mouse should already work, as long as the pane is focused (I understand that's part of the intention of the PR, to scroll where the mouse is positionally, but just stating the size of the win here)

The reason I missed the existing mouse-scrolling functionality is that in iTerm2, “Automatically enable alternate mouse scroll” is disabled by default, which prevents the feature from working. After enabling it, the functionality worked very well.

If enabling the terminal setting is sufficient for the existing functionality to work, then the changes in my PR might be unnecessary.

@joetannenbaum

Copy link
Copy Markdown
Collaborator

Ah ok, that makes sense, didn't realize that was off by default. I think we're probably good here then, thank you again! We can re-visit of you feel something else mouse-related falls short.

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