ArcadeRenegade / SidebarDiagnostics

A simple sidebar for Windows desktop that displays hardware diagnostic information.
GNU General Public License v3.0
2.26k stars 201 forks source link

Ignore ABN_POSCHANGED message if work area didnt change #427

Open Dyrax opened 1 year ago

Dyrax commented 1 year ago

I had a problem where the sidebar would get into an infinite loop of reloading/flickering when using the reserve space setting because every reload would trigger a new ABN_POSCHANGED message which would trigger another reload...

After every reload/repositioning I save the current work area. We can ignore all ABN_POSCHANGED messages unless the work area changes.

This PR also prevents unnecessary reloads when clicking on the edge of an unlocked taskbar.

The async delays in SetAppBar and BindAppBar (Windows.cs) might now be obsolete but I wasn't 100% sure. Removing them worked on my machine.

ArcadeRenegade commented 1 year ago

Ty I'll try to test it on my PC and merge soon