-
Notifications
You must be signed in to change notification settings - Fork 228
Limit monitor-specific scaling to supported autoscale modes #3475
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 failures are due to dependent PR: eclipse-platform/eclipse.platform.swt#2709 not being merged yet. |
28e96c8 to
d41719a
Compare
d41719a to
938b37b
Compare
938b37b to
07f8c15
Compare
.../org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/dialogs/ViewsPreferencePage.java
Outdated
Show resolved
Hide resolved
e03b834 to
3ca1192
Compare
3e07a44 to
ed384a1
Compare
9133cb5 to
b430848
Compare
amartya4256
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.
I have tested the PR and it seems good to me. Approving.
bundles/org.eclipse.ui.workbench/eclipseui/org/eclipse/ui/internal/Workbench.java
Outdated
Show resolved
Hide resolved
b430848 to
06b0ed0
Compare
|
The many test failures seem to be because actually the That will be implicitly fixed with: For that reason, I propose to process that PR first. I wanted to merge it after this one to avoid that a consumer with a to-be-made-incompatible |
e833ea5 to
b0875bd
Compare
b0875bd to
ce23a51
Compare
ce23a51 to
e818531
Compare
With these changes, Eclipse applications will not fail to start anymore when an swt.autoScale value that is incompatible to monitor-specific scaling on Windows is set. The following changes are implemented: - Upon starting an Eclipse application with an swt.autoScale setting that is not compatible with monitor-specific scaling on Windows, the monitor-specific scaling will be disabled and an error dialog will be shown - Users won't be able to enable monitor-specific scaling from appearance page if unsupported swt.autoScale mode is set via system property / ini file
e818531 to
939256f
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.
I have squashed the commits and adapted the commit message to only reflect the changes to Platform UI and not the ones to SWT.
These changes have following affects:
Testing
Scenario#1: Starting application with
-Dswt.autoScale=int200Result: App should start (if monitor-specific scaling is not checked in Appearance page), but option to turn on monitor-specific scaling in Appearance page should be disabled.
Scenario#2: Starting application with
-Dswt.autoScale=int200and-Dswt.autoScale.updateOnRuntime=trueResult: App will start, user will see an error dialog stating the incompatibility of an option and monitor specific-scaling will be disabled.
Scenario#3: Starting application with
-Dswt.autoScale=int200and-Dswt.autoScale.updateOnRuntime=trueand
-Dswt.autoScale.force=trueResult: makes more sense with swt applications. if user really wants to use other scaling mode with monitor-specific scaling.
Scenario#4: Starting application with
-Dswt.autoScale=quarterResult: App should start, and user should be able to turn on/off the monitor-specific scaling in Appearance page.
Note: following is the list of allowed autoscale values to be paired with monitor-specific scaling: ["false", "quarter", "exact"] or any concrete zoom value e.g. 225
Dependency