-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Description
Hi, I've started cooking themes yesterday and decided I should collect the problems I've encountered.
-
1.If an .ini file doesn't start with a definition591dfdd[but an empty line or a comment, an empty theme will be added to the selection menu. -
2.When a tooltip is presented after enabling an option, the text color is hardcoded to be white(-ish)Even more checks and fixes #18800 -
3.Theme system fixes and additions #19984PopupStyleBghas no adequate Foreground option,ItemStyleFgis used instead. -
4.InfoStyleBgseems to be unused (or at least I can't find it anywhere.)is only used in some tabs of System Information. -
5.Headers usually don't have backgrounds but the Collapsible Headers under Control remapping do and useTheme system fixes and additions #19984ItemStyleBg. This is inconsistent behavior. There is also no customizable header background color adequate toHeaderStyleFg.
EDIT: collapsible headers useItemStyleFgandItemStyleBgsince Improve some asserts, minor fixes (theme, warnings) #19547. -
6.Homebrew Store view has hardcoded colors in the title.Better control profile for Retroid devices, minor fixes and cleanup #19550 and Fix minor theme issue in Store #19582 -
7.We should consider resetting the Color Tint and Color Saturation options to their default values whenever a new Theme is selected.Better control profile for Retroid devices, minor fixes and cleanup #19550 -
8.The game list settings popup seems to be another can of worms... featuring the combination ofImprove some asserts, minor fixes (theme, warnings) #19547 and More theming work #19995PopupStyleBg, andHeaderStyleFg. -
9.The amount of background opacity, when the game is paused, is hardcoded. -
10.WhenPopupStyleBgis transparent, there seems to be some radial gradient in play. -
11.The Color Tint and Color Saturation popups remove the background when the game is paused, as if I were selecting shaders in the Display Layout screen. -
12.Tooltips don't have separate colors and reuseThe amount of transparency for hint popup backgrounds seems to be hardcoded. Theme system fixes and additions #19984PopupStyleBg/ItemStyleFg. This can lead to tooltips not being very distinguishable from menu items in some cases, such as with the Dark theme. -
13.The default value ofImprove custom themes #20333ThemeInfo::uPopupStyleBgwas never updated from the old Holo inspired color to the current defaultPopupStyleBg. -
14.The scrollbar is not customizable.Theme system fixes and additions #19984 -
15.The list of themes seems to get updated only when a new theme is selected, rather than when the list pops up. This is noticeable when extending an already existing.inifile with a new theme. (This doesn't happen with shader selection.) -
16.(Not necessarily an issue, but worth noting.) Already existing themes can be overwritten by making the name field of a new theme match the original's. Because Default is both hardcoded and (redundantly) defined indefaultthemes.ini, this can lead to situations like13. -
17.When the last used theme no longer exists when the emulator is started, the first theme from the first file will be loaded, but the option will sayTheme system fixes and additions #19984Default. The proper default theme should be loaded instead. -
18.Rework color usage for sliders. See Theme Related Problems #18802 (comment)Make slider colors themable #20308 -
19.Clean up text input and label styles in popups. See Theme Related Problems #18802 (comment)More theming work #19995 and Exit fixes, AutoDNS fixes #19997 -
20.In the popup themes are listed in the order of the.ini's, rather than alphabetically. (This doesn't happen with shader selection.) -
21.hrydgard proposes making the scrollbar fully opaque when being held down. See Theme Related Problems #18802 (comment)
Let's discuss all of this below.