Skip to content

[win32] Register internal ZoomChange listener for monitor specific scaling#2584

Merged
akoch-yatta merged 1 commit into
eclipse-platform:masterfrom
vi-eclipse:win32-register-zoomchange-listener
Oct 2, 2025
Merged

[win32] Register internal ZoomChange listener for monitor specific scaling#2584
akoch-yatta merged 1 commit into
eclipse-platform:masterfrom
vi-eclipse:win32-register-zoomchange-listener

Conversation

@akoch-yatta
Copy link
Copy Markdown
Contributor

As the DPI change behavior in the windows implementation is fully using the ZoomChange event now, it must be ensured, that the internal ZoomChange listener, that adapt the Wigets for monitor specific scaling must be be registered in this case.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Oct 2, 2025

Test Results

  118 files  +1    118 suites  +1   11m 59s ⏱️ + 3m 45s
4 583 tests ±0  4 547 ✅ ±0  36 💤 ±0  0 ❌ ±0 
  330 runs  ±0    307 ✅ ±0  23 💤 ±0  0 ❌ ±0 

Results for commit 35815eb. ± Comparison against base commit b2b5d45.

This pull request removes 1 and adds 1 tests. Note that renamed tests count towards both.
org.eclipse.swt.widgets.ControlWin32Tests ‑ testScaleFontCorrectlyInNoAutoScaleScenario
org.eclipse.swt.widgets.ControlWin32Tests ‑ testDoNotScaleFontInNoAutoScaleScenario

♻️ This comment has been updated with latest results.

@akoch-yatta akoch-yatta force-pushed the win32-register-zoomchange-listener branch from 04c3f4d to 788a6e6 Compare October 2, 2025 13:09
…aling

As the DPI change behavior in the windows implementation is fully
using the ZoomChange event now, it must be ensured, that the internal
ZoomChange listener, that adapt the Wigets for monitor specific scaling
must be be registered in this case.
@HeikoKlare HeikoKlare force-pushed the win32-register-zoomchange-listener branch from 788a6e6 to 35815eb Compare October 2, 2025 13:28
@akoch-yatta akoch-yatta merged commit 2fde7cc into eclipse-platform:master Oct 2, 2025
18 of 22 checks passed
@akoch-yatta akoch-yatta deleted the win32-register-zoomchange-listener branch October 2, 2025 13:49
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.

2 participants