Skip to content

Add missing descriptions to Colours and Fonts theme entries in Preferences#2453

Merged
elsazac merged 1 commit intoeclipse-platform:masterfrom
elsazac:ColorsandFonts2
Feb 9, 2026
Merged

Add missing descriptions to Colours and Fonts theme entries in Preferences#2453
elsazac merged 1 commit intoeclipse-platform:masterfrom
elsazac:ColorsandFonts2

Conversation

@elsazac
Copy link
Member

@elsazac elsazac commented Feb 9, 2026

Consistent use of descriptions in Colours and Fonts preferences

Some entries in the Colours and Fonts preference page already provide descriptions, while others leave the description area empty. This change makes use of the existing descriptions and applies them to theme entries so that the description box is consistently populated and available descriptions are properly utilised. This improves clarity and aligns the presentation with the approach used with the subtree elements in Eclipse preference pages and gives more clarity on what each entity is used for. The Colours and Fonts preference page shows a description panel which is empty now. There are already descriptions available for some and not available for rest and they are not being used so as to display in the box. This change adds the existing description usage to theme entries so that users always see meaningful information when navigating these settings.

The main change pertaining to this PR has been merged via eclipse-platform/eclipse.platform.ui#3666 and some of the remaining changes are addressed here.

via ThemeElementCategory, while others do not.
This change uses the existing descriptions to display in the description
box and adds description for ones that doesn't have any entries to
ensure the description area is not left empty and that available
descriptions are consistently utilised.
This aligns with the standard usage elsewhere in Eclipse preference
pages.

The main change pertaining to this has been merged in
eclipse.platform.ui via
eclipse-platform/eclipse.platform.ui#3666
and the remaining part is addressed via this change.
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

Test Results

 1 945 files   -  32   1 945 suites   - 32   1h 36m 57s ⏱️ + 1m 16s
 4 743 tests ±  0   4 719 ✅ ±  0   24 💤 ±0  0 ❌ ±0 
13 815 runs   - 414  13 634 ✅  - 413  181 💤  - 1  0 ❌ ±0 

Results for commit 68f7345. ± Comparison against base commit 69f2124.

@elsazac
Copy link
Member Author

elsazac commented Feb 9, 2026

Build looks good, merging this now.

@elsazac elsazac merged commit 17eafa7 into eclipse-platform:master Feb 9, 2026
21 of 22 checks passed
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