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

Dim or otherwise indicate "inactive" editor/terminal #30522

Closed
jasonhulbert opened this issue Jul 12, 2017 · 65 comments · Fixed by #189248 or #190751
Closed

Dim or otherwise indicate "inactive" editor/terminal #30522

jasonhulbert opened this issue Jul 12, 2017 · 65 comments · Fixed by #189248 or #190751
Assignees
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan ux User experience issues
Milestone

Comments

@jasonhulbert
Copy link

One of the best features of VS Code is the built-in terminal. However, I often begin typing in the editor when in fact I think I'm in the terminal - and vice versa. To alleviate this issue, it would be helpful to visually indicate which portion of the IDE is "active" vs. "inactive". A subtle dimming effect might just do the trick.

@vscodebot vscodebot bot added the terminal Integrated terminal issues label Jul 12, 2017
@jasonhulbert
Copy link
Author

Of course, this would be best suited as an option and, since it could be jarring to current users, not the default behavior.

Personally, I could also see this being useful in split-editor views as well.

@Tyriar
Copy link
Member

Tyriar commented Jul 12, 2017

Some things that could help you with this:

  • "terminal.integrated.cursorBlinking": true, this is how I tell where the focus is. The terminal cursor is also hollow when it's not focused which helps.

  • The high contrast theme adds a border around the terminal when it's focused:

    image

    Probably not much of a workaround but worth mentioning 😉

@bpasero @isidorn @alexandrudima what are your thoughts on maybe having an option something like "workbench.focusBorder": "everywhere" which would draw the focus border around whatever is focused. Right not a border is not drawn when focusing the editor and panels. This could be useful to give an extra visual cue on which part of the workbench is focused. It would also be particularly nice when working with the workbench.action.navigate* commands.

We used to show the border around the explorer not that long ago which i actually liked. A border around the terminal and editor would be nice too.

Alternatively an option to dim the non-active view as proposed by the OP.

@Tyriar Tyriar added under-discussion Issue is under discussion for relevance, priority, approach workbench and removed terminal Integrated terminal issues labels Jul 12, 2017
@Tyriar Tyriar added this to the Backlog milestone Jul 12, 2017
@jasonhulbert
Copy link
Author

jasonhulbert commented Jul 12, 2017

@Tyriar thanks for the response!

I do have blinking cursor set currently and when I'm on my macbook it's usually in a close enough proximity (visually) so that I see it but when I'm on my two large external displays at fullscreen it's easy to miss the blinking cursor if you're looking elsewhere and working quickly.

The high contrast mode makes my eyes bleed haha. None the less, that's great to point out here.

+1 on the focus border you have suggested.

Thanks again

@bpasero
Copy link
Member

bpasero commented Jul 12, 2017

I do not like the border (in fact I even get annoyed by our blue focus ring so I disabled it via theme customization). I would prefer a solution that does not introduce ugly borders.

@jasonhulbert
Copy link
Author

jasonhulbert commented Jul 12, 2017

@bpasero What are you thoughts on the dimming effect? Possibly with an optional "amount"?

iTerm is an example of this:

screen shot 2017-07-12 at 1 14 52 pm

@Tyriar
Copy link
Member

Tyriar commented Jul 12, 2017

I prefer borders personally 😛

image

@bpasero
Copy link
Member

bpasero commented Jul 12, 2017

Yeah dimming might be better. We have this problem already today where when having multiple editors open side by side it is not clear which editor is active.

@bpasero bpasero added ux User experience issues and removed workbench labels Nov 16, 2017
@Tyriar Tyriar removed their assignment Dec 23, 2017
@weinand weinand added the feature-request Request for new features or functionality label Dec 23, 2017
@GregBorrelly
Copy link

is anyone currently working on this request? I'm interested in implementing it.

@Tyriar
Copy link
Member

Tyriar commented Jan 18, 2018

@GregBorrelly I'm not sure there is clear way forward right now.

@rifelpet
Copy link

rifelpet commented Feb 26, 2018

@Tyriar I experience a similar issue as well, specifically when switching between windows. I'll have my web browser "active" and then click on the terminal area of my VS Code window expecting the cursor to be in the terminal so I can start typing commands, however the cursor is unexpectedly placed in the editor. I have to click the terminal area a second time. It'd be great if clicking on the terminal area of an inactive VS Code window would make it active with the cursor in the terminal rather than the editor. Is that a reasonable fix? or should I open a separate issue for this suggestion

@safield
Copy link

safield commented Oct 1, 2018

+1 for dimming the inactive editor/terminal windows.

I always edit in split window mode, and I would really like the inactive windows to be dimmed.

I have been using this plugin for sublime text 3 for 3 years now, and it has convinced me that subtle, full pane dimming, is the most effective way to emphasise or de-emphasise an editor pane.

https://github.com/SublimeText/InactivePanes

@jcherven
Copy link

I also usually edit with a split editor and one or more split terminals, using homerow keybindings to navigate around them. While my color theme (spacemacs dark)'s dimmed tab colors, a hard-blinking block cursor, and the active line indicator does give me some indication of which split is active, I'm still often mistakenly typing into the wrong pane because these subtle cues take a lot of attention to notice.

I really like iTerm2's adjustable inactive pane dimming, it's implemented very well in my opinion. When I have some time I'd like to see if this effect can be hacked in the color theme. Anyone try this yet?

@kwonoj
Copy link

kwonoj commented Apr 11, 2019

I was bugged by this too, while I wasn't able to make take care of terminal pane did a naive impl for text document at least: https://marketplace.visualstudio.com/itemdetails?itemName=ojkwon.vscode-activedocumentindicator until this feature lands in editor.

@paladinu
Copy link

This frustrates me at least a few times a day. Dimming by a configurable amount seems like the clear winner. We should have this by now. Do we vote on it somewhere?

@ruudhanegraaf
Copy link

ruudhanegraaf commented Jan 10, 2020

Throwing in my vote here...
+1

Mainly the terminal window. I can't count the times anymore I've mistakenly typed and overwritten stuff in the editor window, instead of the terminal window.

@jasonwilliams
Copy link
Contributor

This frustrates me at least a few times a day. Dimming by a configurable amount seems like the clear winner. We should have this by now. Do we vote on it somewhere?

Someone needs to re-open this issue as a feature request so that it gets the whole voting thing.
or maybe one of the VSCode staff can update this issue.

@alexr00 ?

@Tyriar
Copy link
Member

Tyriar commented Feb 21, 2020

@jasonhulbert it's in the backlog and unlocked so it can be voted on just fine currently?

@jasonhulbert
Copy link
Author

jasonhulbert commented Feb 21, 2020

Sorry @Tyriar - not sure I understand the question.

If you're asking if my original comment is descriptive enough to be voted on then yes, I think it is accurate enough. I'd be happy to revise or provide more detail if needed.

@jasonhulbert
Copy link
Author

Specifically, folks should take note of the first two comments in this issue and the image/example of iTerm2 shown here #30522 (comment)

@paladinu
Copy link

I think @Tyriar was acutally responding to @jasonwilliams not you @jasonhulbert . Are issues that have been around for a while less likely to be voted on?

@Tyriar
Copy link
Member

Tyriar commented Aug 18, 2023

Thanks for the discussion all, I tweaked the settings in #190751.

@VSCodeTriageBot VSCodeTriageBot added unreleased Patch has not yet been released in VS Code Insiders insiders-released Patch has been released in VS Code Insiders and removed unreleased Patch has not yet been released in VS Code Insiders labels Aug 18, 2023
@lobsterkatie
Copy link

Yay! Thanks, @Tyriar, for taking our input!

@abhijit-chikane
Copy link
Contributor

abhijit-chikane commented Aug 22, 2023

I just checked
The issue with this is If I enable this setting and If I do some changes in editor and click on other file from the explorer it just move the focus from the editor

I don't think we want this behaviour
I just care about the terminal and editor
If I jump in between those then only it should inactive the corrosponding part
but if it is going move the focus even if I click on the file explorer doesn't going to help much atleast for me
@Tyriar

@Tyriar
Copy link
Member

Tyriar commented Aug 22, 2023

@abhijit-chikane I think you're asking for us to track whether the terminal or editor was focused last which is more state than I'd prefer to track for this feature. I'll wait for more feedback on the topic to see the reception as is first.

@lobsterkatie
Copy link

The issue with this is If I enable this setting and If I do some changes in editor and click on other file from the explorer it just move the focus from the editor

Oh, damn, I hadn't even thought of this. Yeah, that's not exactly how I was picturing it, either. That said, @abhijit-chikane, I think it's worth turning it on and living with it for a few weeks. It may turn out to be intuitive, or it may not. And if it's not, we can always open another feature request! 😛

@Tyriar
Copy link
Member

Tyriar commented Aug 22, 2023

and click on other file from the explorer it just move the focus from the editor

Also a little tip, you can double click to select the file in the explorer and focus the editor

@Tyriar
Copy link
Member

Tyriar commented Aug 29, 2023

FYI I'm planning on moving this back to live under accessibility., see #191618 and some discussion about the feature in #191610

@Tyriar
Copy link
Member

Tyriar commented Aug 30, 2023

There's been a bunch of feedback from the team during the test round. I'll be announcing the feature in the release notes' experimental section and this is the plan for upcoming improvements #191775. Feedback is welcome of course!

@lobsterkatie
Copy link

There's been a bunch of feedback from the team during the test round.

I like your system of making an issue specifically for internal testing and then linked issues for feedback! I may suggest it to our product team. 🙂

@Tyriar
Copy link
Member

Tyriar commented Aug 30, 2023

@lobsterkatie this is our "endgame" week where the whole team turn into testers for a bit https://github.com/microsoft/vscode/wiki/Development-Process#end-game, it certainly works pretty well when building devtools 👍

@lobsterkatie
Copy link

lobsterkatie commented Aug 30, 2023

Nice - I didn’t realize it was documented. Will definitely pass that along, thanks! And we do indeed make a devtool - you should check us out sometime!

@Tyriar
Copy link
Member

Tyriar commented Aug 31, 2023

@lobsterkatie ah I think I've heard ads for Sentry and been at the same conferences. I remember seeing https://blog.sentry.io/three-reasons-to-meat-us-at-github-universe/ 😉

@lobsterkatie
Copy link

Oh, how funny, and man, what a throwback! I remember the day our head of design was figuring out how to make the "Sentry" on that hot dog swag.

3DAB48DE-AD26-44B2-A627-21EB962F7456_1_105_c

In any case, we have an electron SDK as well as freestanding browser and node SDKs (which, full disclosure, I helped maintain for a few years) should y'all ever be interested. 🙂

Tyriar added a commit to microsoft/vscode-docs that referenced this issue Sep 6, 2023
Krzysztof-Cieslak pushed a commit to githubnext/vscode that referenced this issue Sep 18, 2023
Krzysztof-Cieslak pushed a commit to githubnext/vscode that referenced this issue Sep 18, 2023
Tyriar added a commit to microsoft/vscode-docs that referenced this issue Sep 28, 2023
@github-actions github-actions bot locked and limited conversation to collaborators Oct 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan ux User experience issues
Projects
None yet