Enum settings should be ordered better #12037

Open
opened 2026-01-31 03:04:38 +00:00 by claunia · 0 comments
Owner

Originally created by @carlos-zamora on GitHub (Jan 12, 2021).

Originally assigned to: @carlos-zamora on GitHub.

Description

In the Settings UI, settings that are stored as enums and are represented as ComboBoxes/RadioButtons are not ordered properly:

  • "none" and "hidden" should always be shown last
  • FontWeight should be in thickness order (with "Custom" being last), not alphabetical order

Technical Details

#8086 introduced a way to leverage the JSON_MAPPINGS (json string to enum setting mapping) as a way to expose that mapping into the TerminalSettingsEditor. Dustin reported that it seems that they are ordered according to the json strings being alphabetical, not the localized Settings UI string being alphabetical.

We probably need some kind of post-processing to establish a good ordering in the TerminalSettingsEditor or the EnumMapping in TerminalSettingsModel.

Originally created by @carlos-zamora on GitHub (Jan 12, 2021). Originally assigned to: @carlos-zamora on GitHub. ## Description In the Settings UI, settings that are stored as enums and are represented as ComboBoxes/RadioButtons are not ordered properly: - "none" and "hidden" should always be shown last - FontWeight should be in thickness order (with "Custom" being last), not alphabetical order ## Technical Details #8086 introduced a way to leverage the `JSON_MAPPINGS` (json string to enum setting mapping) as a way to expose that mapping into the TerminalSettingsEditor. Dustin reported that it seems that they are ordered according to the json strings being alphabetical, _not_ the localized Settings UI string being alphabetical. We probably need some kind of post-processing to establish a good ordering in the TerminalSettingsEditor or the `EnumMapping` in TerminalSettingsModel.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#12037