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

[Peek] WindowActivationState checks are added for focus and close aft… #26364

Conversation

gokcekantarci
Copy link
Contributor

@gokcekantarci gokcekantarci commented May 26, 2023

…er losing focus.

Summary of the Pull Request

PR Checklist

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

@crutkas
Copy link
Member

crutkas commented May 26, 2023

Do we having a tracking issue for this

@gokcekantarci
Copy link
Contributor Author

Do we having a tracking issue for this

A new issue opened and added to PR.

@jaimecbernardo jaimecbernardo added the Hot Fix Items we will product an out-of-band release for label May 26, 2023
@jaimecbernardo
Copy link
Collaborator

I'll be adding a Setting for enabling/disabling this behavior.

@jaimecbernardo
Copy link
Collaborator

Also need to check if this doesn't break #26349

@SamChaps
Copy link
Contributor

I'll be adding a Setting for enabling/disabling this behavior.

I am also in favor of adding a Setting light dismiss. It can be very frustrating especially since it is not a window's normal behavior.

@jaimecbernardo
Copy link
Collaborator

I've added a setting for this, off by default:
image

Also added a Service to the Peek.UI to get the settings, now that some settings influence it.

Copy link
Collaborator

@stefansjfw stefansjfw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested. Code looks good

@jaimecbernardo jaimecbernardo merged commit 0c69e34 into microsoft:main May 30, 2023
jaimecbernardo added a commit that referenced this pull request May 30, 2023
@jaimecbernardo jaimecbernardo mentioned this pull request May 30, 2023
11 tasks
jaimecbernardo added a commit that referenced this pull request May 30, 2023
* catch exception

* Check count of items to avoid the exception being thrown

* Fix regression from #26364

---------

Co-authored-by: Jaime Bernardo <jaime@janeasystems.com>
BLM16 pushed a commit to BLM16/PowerToys that referenced this pull request Jun 22, 2023
* [Peek] WindowActivationState checks are added for focus and close after losing focus.

* Add setting to activate the behavior

---------

Co-authored-by: Jaime Bernardo <jaime@janeasystems.com>
BLM16 pushed a commit to BLM16/PowerToys that referenced this pull request Jun 22, 2023
…26470)

* catch exception

* Check count of items to avoid the exception being thrown

* Fix regression from microsoft#26364

---------

Co-authored-by: Jaime Bernardo <jaime@janeasystems.com>
@gokcekantarci gokcekantarci deleted the Peek]-Losing-Focus-should-close-automatically branch July 7, 2023 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hot Fix Items we will product an out-of-band release for
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants