-
Notifications
You must be signed in to change notification settings - Fork 188
Limit autoscale mode to quarter and exact only #2771
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
Conversation
Test Results 147 files ±0 147 suites ±0 21m 44s ⏱️ -47s Results for commit 9e0a902. ± Comparison against base commit 407cb9a. This pull request removes 3 tests.♻️ This comment has been updated with latest results. |
4ee56bc to
e66f2c4
Compare
bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/internal/DPIUtil.java
Outdated
Show resolved
Hide resolved
edbb7e1 to
a1581ca
Compare
bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/internal/DPIUtil.java
Outdated
Show resolved
Hide resolved
bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/internal/DPIUtil.java
Outdated
Show resolved
Hide resolved
bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/internal/DPIUtil.java
Outdated
Show resolved
Hide resolved
bundles/org.eclipse.swt/Eclipse SWT/common/org/eclipse/swt/internal/DPIUtil.java
Outdated
Show resolved
Hide resolved
5f06264 to
3d00780
Compare
3d00780 to
04fea49
Compare
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.
04fea49 to
9e0a902
Compare
HeikoKlare
left a comment
There was a problem hiding this 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.
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