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

One string fix - classicSwitcher.js for alt+tab icon click event #12430

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

trumenov
Copy link

@trumenov trumenov commented Oct 4, 2024

Describe bug: When you click on the icon on pressed alt+tab - active became focused icon, but not you clicked.
This PR will fix this small bug.

@trumenov trumenov changed the title Update classicSwitcher.js Update classicSwitcher.js for alt+tab icon click event Oct 4, 2024
@trumenov trumenov changed the title Update classicSwitcher.js for alt+tab icon click event One string - classicSwitcher.js for alt+tab icon click event Oct 8, 2024
@trumenov trumenov changed the title One string - classicSwitcher.js for alt+tab icon click event One string fix - classicSwitcher.js for alt+tab icon click event Oct 8, 2024
@mtwebster
Copy link
Member

Can you describe the issue in more detail and steps to reproduce?

Currently:

  • Alt-tab brings up the list
  • Clicking on icon A makes that window A focused and closes the alt-tab switcher.
  • Alt-tabbing brings up the list again, with icon A as the left-most icon, and the one next to it highlighted (because it will be the window made active if you release alt-tab immediately).

This seems correct to me. Is it possible when you alt-tab a second time your mouse pointer is (randomly) in a position that causes a different icon to be highlighted when the icons appear?

@mtwebster mtwebster added the Blocked Needs rebase, changes, or discussion label Nov 25, 2024
@clefebvre clefebvre self-assigned this Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Blocked Needs rebase, changes, or discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants