Skip to content

Hide and immediately re-show window when cloaking to manage focus #40431

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

Conversation

jiripolasek
Copy link
Contributor

@jiripolasek jiripolasek commented Jul 6, 2025

Summary of the Pull Request

When the palette loses focus, the window is hidden and then immediately shown again while still cloaked (invisible). This sequence ensures that focus is correctly handed off to the next window in the Z-order. It prevents the invisible palette from intercepting input and keeps the palette ready for the next activation.

PR Checklist

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

When the palette loses focus, the window is hidden and then immediately shown again while still cloaked (invisible).
This sequence ensures that focus is correctly handed off to the next window in the Z-order.
It prevents the invisible palette from intercepting input and keeps the palette ready for the next activation.

This comment has been minimized.

@jiripolasek
Copy link
Contributor Author

Closing. Succeeded by #40445.

@jiripolasek jiripolasek closed this Jul 9, 2025
zadjii-msft added a commit that referenced this pull request Jul 9, 2025
this is a port of ce15032 onto main, for just the FG change.

When we cloak our window, we want to make sure to _manually_ give FG
back to another window. Because apparently cloaked windows can have FG.
beacause that makes sense 🤦

Closes #39638
supersedes #40431 

Co-authored-by: jiripolasek <me@jiripolasek.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Product-Command Palette Refers to the Command Palette utility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants