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

Check if a restored window is active #28

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

imawizard
Copy link
Owner

@imawizard imawizard commented Jun 22, 2024

For some windows like Windows Explorer there seems to be no focus event when restoring them from e.g. a hidden state.
So when a window is restored, check whether it's the active one and if so treat the restore event like a focus.

@imawizard imawizard changed the title fix(focusindicator): Don't update for unmanaged windows that are invi… fix: Check if window is active on EV_WINDOW_RESTORED Jun 22, 2024
@imawizard imawizard added the bug Something isn't working label Jun 22, 2024
@imawizard imawizard changed the title fix: Check if window is active on EV_WINDOW_RESTORED Check if window is active on EV_WINDOW_RESTORED Jun 22, 2024
@imawizard imawizard changed the title Check if window is active on EV_WINDOW_RESTORED Check restored windows for focus Jun 22, 2024
For some windows like Windows Explorer there seems to be no focus event
when restoring them from e.g. a hidden state.
So when a window is restored, check whether it's the active one and if so
treat the restore event like a focus.
@imawizard imawizard force-pushed the fixes/focus-restored-window branch from 7529d82 to 19783e4 Compare June 22, 2024 20:30
@imawizard imawizard changed the title Check restored windows for focus Check if a restored window is active Jul 30, 2024
@imawizard imawizard marked this pull request as ready for review July 30, 2024 11:17
@imawizard imawizard merged commit accfd1c into ahk-v2 Jul 30, 2024
1 check passed
@imawizard imawizard deleted the fixes/focus-restored-window branch July 30, 2024 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant