Skip to content

Set proper default for monitor-specific scaling preference#4005

Open
HeikoKlare wants to merge 1 commit into
eclipse-platform:masterfrom
HeikoKlare:monitor-specific-scaling-restore-default
Open

Set proper default for monitor-specific scaling preference#4005
HeikoKlare wants to merge 1 commit into
eclipse-platform:masterfrom
HeikoKlare:monitor-specific-scaling-restore-default

Conversation

@HeikoKlare
Copy link
Copy Markdown
Contributor

When restoring the default values for the "Appearance" preference page, the Windows-specific preference for enabling/disabling monitor-specific scaling is not updated. The default value for this preference is currently defined at every place accessing the preference redundantly.

With this change, a proper default value in the default preference scope is defined for the monitor-specific scaling preference. This value is used by all consumers of the preference via a ScopedPreferenceStore. The defaults restoration of the "Appearance" preference page is extended with a restore of the monitor-specific scaling preference.

How to test

Open the "Apperance" page of the preference dialog and press "Restore Defaults". Without this change, the "Use monitor-specific UI scaling" option will not change. With this change, the default value (enabled) will be applied.

image

When restoring the default values for the "Appearance" preference page,
the Windows-specific preference for enabling/disabling monitor-specific
scaling is not updated. The default value for this preference is
currently defined at every place accessing the preference redundantly.

With this change, a proper default value in the default preference scope
is defined for the monitor-specific scaling preference. This value is
used by all consumers of the preference via a ScopedPreferenceStore. The
defaults restoration of the "Appearance" preference page is extended
with a restore of the monitor-specific scaling preference.
@github-actions
Copy link
Copy Markdown
Contributor

Test Results

   867 files  ±0     867 suites  ±0   56m 40s ⏱️ - 5m 7s
 7 988 tests ±0   7 745 ✅ ±0  243 💤 ±0  0 ❌ ±0 
20 466 runs  ±0  19 811 ✅ ±0  655 💤 ±0  0 ❌ ±0 

Results for commit fdfa795. ± Comparison against base commit b58e3b4.

@HeikoKlare HeikoKlare marked this pull request as ready for review May 15, 2026 12:15
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.

1 participant