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

Fixed problem that caused suspended modern Win10 apps to be managed #3

Merged
merged 3 commits into from
May 2, 2020

Conversation

nir9
Copy link
Contributor

@nir9 nir9 commented Apr 24, 2020

Problem:
Windows 10 modern apps invisible windows are randomly appearing every hour or so and taking real estate from dwm-win32 tiles.
Cause:
Windows 10 loads the apps in suspended state to optimize their load time when they are actually needed
Solution:
Make dwm-win32 identify these "cloaked" windows and not manage them

@prabirshrestha
Copy link
Owner

This seems to completely ignore windows 10 modern apps such as Calculator. would it be possible to not ignore those?

Can you send the debugger commit in different PR so I can merge it.

@nir9
Copy link
Contributor Author

nir9 commented Apr 25, 2020

Ok, I see what you are talking about, it looks like the problem is dwm-win32 getting the "new window created" notification while the modern app is still "cloaked" (there is probably some kind of race here).
So I'll try to work on a different solution.

@prabirshrestha
Copy link
Owner

one option is to use this to check which windows are active.

https://www.nirsoft.net/utils/winlister.html

@nir9
Copy link
Contributor Author

nir9 commented Apr 29, 2020

WinLister does not solve the problem since it also shows the suspended modern apps as visible windows, and also it is closed source

@nir9
Copy link
Contributor Author

nir9 commented May 2, 2020

@prabirshrestha Fixed the problem, now it catches modern apps correctly once they are uncloaked

@prabirshrestha prabirshrestha merged commit 7c81e2a into prabirshrestha:master May 2, 2020
@prabirshrestha
Copy link
Owner

Merged. Thanks for the fix. It has been irritating me too.

Also thanks for fixing list issues. I really need to spend a bit more time to add github actions on some of these.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants