-
Notifications
You must be signed in to change notification settings - Fork 28.7k
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
💬 Add PID and executable/arguments to terminal tab hover #171258
💬 Add PID and executable/arguments to terminal tab hover #171258
Conversation
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
Signed-off-by: Babak K. Shandiz <babak.k.shandiz@gmail.com>
@Tyriar Sorry, I just pushed a commit to escape possible markdown tokens in the arguments list. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @babakks, I pushed a change to pipe the actual args through if they get tweaked on the pty host and also tweaked the message:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure what's causing the failure 🤔
It was causing bad wrapping and a really wide hover
Fixes #170580
I think this can be a good starter. It adds the PID and executable/arguments to terminal tab hovers.
Note that, I took the quoting of the arguments from here. However, I'm not sure if it works with all special cases (e.g., those having a single quote character within the arg values).
Here are some screen captures of the updated hovers:
(Tabbed, without arguments)
(Tabbed, with arguments)
(Single tab, without arguments)