Add missing descriptions to Colours and Fonts theme entries in Preferences#3666
Merged
vogella merged 1 commit intoeclipse-platform:masterfrom Feb 3, 2026
Merged
Conversation
Contributor
Contributor
|
Good one, thanks for improving UI experience with such small improvements. |
elsazac
force-pushed
the
ColorsandFonts1
branch
from
b2af547 to
b213e14
Compare
Member
Author
|
@vogella would you be able to review and approve this? and I would like to merge this if there are no outstanding concerns on this. |
elsazac
force-pushed
the
ColorsandFonts1
branch
from
b213e14 to
2578029
Compare
Contributor
|
@BeckerWdf can you review? |
vogella
force-pushed
the
ColorsandFonts1
branch
from
2578029 to
765d754
Compare
elsazac
force-pushed
the
ColorsandFonts1
branch
from
765d754 to
adecc5c
Compare
Some Colors and Fonts entries already provide user-facing descriptions 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.
elsazac
force-pushed
the
ColorsandFonts1
branch
from
adecc5c to
135b749
Compare
Contributor
|
Thanks @elsazac |
elsazac
added a commit
to elsazac/eclipse.platform
that referenced
this pull request
Feb 9, 2026
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.
elsazac
added a commit
to eclipse-platform/eclipse.platform
that referenced
this pull request
Feb 9, 2026
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.
elsazac
added a commit
to elsazac/eclipse.jdt.ui
that referenced
this pull request
Feb 13, 2026
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. The main change pertaining to this has been merged in eclipse.platform.ui and eclipse.platform via eclipse-platform/eclipse.platform.ui#3666 and eclipse-platform/eclipse.platform#2453 respectively. One of the remaining part pertaining to the above PR's are addressed via this change.
3 tasks
noopur2507
pushed a commit
to eclipse-jdt/eclipse.jdt.ui
that referenced
this pull request
Feb 13, 2026
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. The main change pertaining to this has been merged in eclipse.platform.ui and eclipse.platform via eclipse-platform/eclipse.platform.ui#3666 and eclipse-platform/eclipse.platform#2453 respectively. One of the remaining part pertaining to the above PR's are addressed via this change.
eclipse-egit-bot
pushed a commit
to eclipse-egit/egit
that referenced
this pull request
Feb 17, 2026
Some Colors and Fonts entries already provide descriptions 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 that the description area is not left empty and that available descriptions are consistently utilised. The main change pertaining to this has been merged in eclipse.platform.ui and eclipse.platform via eclipse-platform/eclipse.platform.ui#3666, eclipse-platform/eclipse.platform#2453, and eclipse-jdt/eclipse.jdt.ui#2799 respectively. some of the remaining part pertaining to the above PR's is addressed via this change. Change-Id: Ib3b195cfc9978557d7c18545f6e3c83d1154392a
elsazac
added a commit
to elsazac/www.eclipse.org-eclipse
that referenced
this pull request
Feb 19, 2026
elsazac
added a commit
to elsazac/www.eclipse.org-eclipse
that referenced
this pull request
Feb 19, 2026
merks
pushed a commit
to eclipse-platform/www.eclipse.org-eclipse
that referenced
this pull request
Feb 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
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 PR make use of the description. This change adds the existing description usage to theme entries so that users always see meaningful information when navigating these settings.
for eg:
CSS bridge demotheme andview and editor foldershave descriptions available but it isn't displayed in the box; This PR makes changes to utilise the description availablebefore :
after :
similarly, with entities
BasicandView and Editor foldersThere are few more entries coming from
eclipse.platformthat needs similar updates. Once this change goes in, I plan to follow up on those as well.