Skip to content

Conversation

@ShahzaibIbrahim
Copy link
Contributor

@ShahzaibIbrahim ShahzaibIbrahim commented Nov 12, 2025

swt applications will be limited to use autoscale modes quarter and exact only otherwise user will see an error message stating the incompatibility. Also the tests testing other autoscale values have been removed.

This PR will further affect: eclipse-platform/eclipse.platform.ui#3475

@github-actions
Copy link
Contributor

github-actions bot commented Nov 12, 2025

Test Results

  147 files  ±0    147 suites  ±0   21m 44s ⏱️ -47s
4 670 tests  - 3  4 648 ✅  - 3  22 💤 ±0  0 ❌ ±0 
  411 runs   - 3    406 ✅  - 3   5 💤 ±0  0 ❌ ±0 

Results for commit 9e0a902. ± Comparison against base commit 407cb9a.

This pull request removes 3 tests.
org.eclipse.swt.widgets.ControlWin32Tests ‑ testAutoScaleImageData(float, String, boolean)[6]
org.eclipse.swt.widgets.ControlWin32Tests ‑ testAutoScaleImageData(float, String, boolean)[7]
org.eclipse.swt.widgets.ControlWin32Tests ‑ testAutoScaleImageData(float, String, boolean)[8]

♻️ This comment has been updated with latest results.

@ShahzaibIbrahim ShahzaibIbrahim force-pushed the master-399-p2 branch 2 times, most recently from 4ee56bc to e66f2c4 Compare November 17, 2025 11:40
@ShahzaibIbrahim ShahzaibIbrahim marked this pull request as ready for review November 17, 2025 11:40
@ShahzaibIbrahim ShahzaibIbrahim force-pushed the master-399-p2 branch 2 times, most recently from edbb7e1 to a1581ca Compare December 3, 2025 12:54
@ShahzaibIbrahim ShahzaibIbrahim force-pushed the master-399-p2 branch 2 times, most recently from 5f06264 to 3d00780 Compare December 4, 2025 12:06
SWT applications will be limited to use autoscale modes quarter and
exact when having monitor-specific scaling enabled. With other,
unsupported values the application will not start. To enforce
application start with an incompatible autoscale mode, a new "force"
value for the swt.autoScale.updateOnRuntime property is introduced. Also
the tests testing other autoscale values have been
removed.
Copy link
Contributor

@HeikoKlare HeikoKlare left a comment

Choose a reason for hiding this comment

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

This looks fine now. The change also implicitly fices a wrong handling of a default swt.autoScale value when using monitor-specific scaling (see eclipse-platform/eclipse.platform.ui#3475 (comment)). Thus proposing to merge this before eclipse-platform/eclipse.platform.ui#3475 to fix the adopted method first.

@HeikoKlare HeikoKlare merged commit 5071736 into eclipse-platform:master Dec 5, 2025
28 of 30 checks passed
@HeikoKlare HeikoKlare deleted the master-399-p2 branch December 5, 2025 13:36
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.

Limit monitor-specific scaling to supported autoscale modes

3 participants