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

Destroy unused handles on WM_DISPLAYCHANGE #128

Open
fedejeanne opened this issue Oct 11, 2024 · 2 comments
Open

Destroy unused handles on WM_DISPLAYCHANGE #128

fedejeanne opened this issue Oct 11, 2024 · 2 comments
Labels
HiDPI A HiDPI-Related Issue or Feature SWT Issue for SWT
Milestone

Comments

@fedejeanne
Copy link

Issue

Currently, the OS handles for resources in an unnecessary zoom level are being removed when the OS-event WM_DPICHANGED is triggered (see eclipse-platform/eclipse.platform.swt#1477). This poses a problem because this event is only triggered in 2 situations:

  1. The window is dragged to a monitor with different scaling
  2. The monitor where the window is located changes its scaling

But there is one scenario when this event is not triggered, that is:
3. A monitor that does not contain the window changes its scaling.

Goal

Do a PoC and see if it's possible to react to another event and trigger the cleanup: WM_DISPLAYCHANGE

@fedejeanne fedejeanne converted this from a draft issue Oct 11, 2024
@fedejeanne fedejeanne added SWT Issue for SWT HiDPI A HiDPI-Related Issue or Feature labels Oct 11, 2024
@fedejeanne
Copy link
Author

@amartya4256 fyi

@fedejeanne fedejeanne changed the title Destroy unusued handles on WM_DISPLAYCHANGE Destroy unused handles on WM_DISPLAYCHANGE Oct 11, 2024
@akoch-yatta akoch-yatta added this to the 4.35 M1 milestone Nov 8, 2024
@HeikoKlare
Copy link
Contributor

First to pick as slack for M1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
HiDPI A HiDPI-Related Issue or Feature SWT Issue for SWT
Projects
Status: 🔖 Ready: Atomic
Development

No branches or pull requests

3 participants