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

Unfocused editors should regain focus (be un-dimmed) on scroll #191671

Open
bhavyaus opened this issue Aug 29, 2023 · 5 comments
Open

Unfocused editors should regain focus (be un-dimmed) on scroll #191671

bhavyaus opened this issue Aug 29, 2023 · 5 comments
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug
Milestone

Comments

@bhavyaus
Copy link
Collaborator

Testing #191527

In the gif, you can see that after picking a file to be opened in the editor, I can scroll in the dimmed editor. This feels weird to me.

Screen.Recording.2023-08-29.at.10.52.32.AM.mov
@Tyriar Tyriar added bug Issue identified by VS Code Team member as probable bug accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues labels Aug 29, 2023
@Tyriar Tyriar added this to the Backlog milestone Aug 29, 2023
@Tyriar
Copy link
Member

Tyriar commented Aug 29, 2023

Right now the feature uses a set of CSS rules, I don't think it would be possible to fix this problem without hooking up some events. For example if we could listen to an editor scroll event and treat it as focused for x seconds after that?

@lobsterkatie
Copy link

I can scroll in the dimmed editor. This feels weird to me.

FWIW, this matches the native behavior in OS X. (Notice how I can scroll in the other Chrome window, even though the one with this issue in it is in front and focused.) I thought it was a little weird when I first discovered it in OS X, too, but you get used to it pretty quickly.

Screen.Recording.2023-08-30.at.11.28.20.AM.mov

@Destroy666x
Copy link

Destroy666x commented Sep 10, 2023

It's not the scrolling part which is weird. That's normal in all popular window systems. The dimming part is quite annoying though. It's clear you're using that part of the window and want to see the contents better, IMO.

So I don't think it should necessarily "regain focus" as the issue suggest, but more like "regain brightness". If not by default then with a "Scrolling unfocused editor disables dimming" kind of setting.

@lobsterkatie
Copy link

It's not the scrolling part which is weird. That's normal in all popular window systems. The dimming part is quite annoying though.

Ah - that makes more sense. Thanks for clarifying.

I don't think it should necessarily "regain focus" as the issue suggest, but more like "regain brightness". If not by default then with a "Scrolling unfocused editor disables dimming" kind of setting.

Could be a setting, but FWIW, now that I've started using the feature, I like the way it works currently. The high-level objective of the feature to is make it clear to the user which view will be affected by keyboard input (so I can stop typing git push into random files, for example), and the answer to that question doesn't change if I scroll an unfocused view.

@JimGitFE
Copy link

JimGitFE commented Dec 2, 2023

In case someone is wondering how to disable Dim Unfocused

  1. Open the Command Palette Ctrl + Shift + P (Windows/Linux)
  2. Type Open Accessibility Settings
  3. Uncheck Accesibility > Dim Unfocused

As per vscode August Release

"accessibility.dimUnfocused.enabled": false,
"accessibility.dimUnfocused.opacity": 1,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug
Projects
None yet
Development

No branches or pull requests

5 participants