HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. "Read more...":https://support.plan.io/news/187
Bug #853130
closed
Qt client configuration can end up with illegal theme "Freeciv", leading to display problems
0%
Description
- Client rc file somehow ends up with gui_qt_default_theme_name="Freeciv", which is not a valid theme name;
- Qt client reacts to this illegal theme name by partially loading UI-theme-related material, leading to bad display.
This may have been exposed or made worse by fixes for #782349 and/or #850290, between 2.6.0 and 2.6.1.
It can be worked around by explicitly setting the theme in the client options. (The illegal theme name is not visible in the UI -- the UI state is indistinguishable from having the default "NightStalker" theme selected -- but re-selecting NightStalker and saving options is sufficient to fix the symptom.)
Files
| qt_badtheme_start.png (220 KB) qt_badtheme_start.png | 2.6.1 Qt client with "Freeciv" in rc file: start screen | ||
| qt_badtheme_main.png (202 KB) qt_badtheme_main.png | 2.6.1 Qt client with "Freeciv" in rc file: main game screen | ||
| qt_badtheme_config.png (31.4 KB) qt_badtheme_config.png | 2.6.1 Qt client with "Freeciv" in rc file: client options UI | ||
| qt_goodtheme_start.png (337 KB) qt_goodtheme_start.png | 2.6.1 Qt client with "NightStalker" in rc file: start screen | ||
| qt_goodtheme_main.png (226 KB) qt_goodtheme_main.png | 2.6.1 Qt client with "NightStalker" in rc file: main game screen | ||
| qt_goodtheme_config.png (33.7 KB) qt_goodtheme_config.png | 2.6.1 Qt client with "NightStalke" in rc file: client options UI | ||
| 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch (1.42 KB) 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch | |||
| 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch (1.16 KB) 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch |
Related issues
Updated by Jacob Nevins over 3 years ago
- File qt_badtheme_start.png qt_badtheme_start.png added
- File qt_badtheme_main.png qt_badtheme_main.png added
- File qt_badtheme_config.png qt_badtheme_config.png added
- File qt_goodtheme_start.png qt_goodtheme_start.png added
- File qt_goodtheme_main.png qt_goodtheme_main.png added
- File qt_goodtheme_config.png qt_goodtheme_config.png added
Observations of the symptom, point 2:
The attached screenshots compare what happens when gui_qt_default_theme_name="Freeciv" ("badtheme") and gui_qt_default_theme_name="NightStalker" ("goodtheme"). Things that go wrong include:- No theming on start screen
- In main game screen:
- Buttons on left go mostly white when you hover over them
- Chat window has white background instead of black
- White-on-light-grey text in some window titles
Updated by Jacob Nevins over 3 years ago
How we end up with the invalid theme name in config, point 1:
I think it is due to migration of settings from 2.5. Somehow, I ended up with gui_qt_default_theme_name="Freeciv" in my .freeciv-client-rc-2.5; I think this was migrated to .freeciv/freeciv-client-rc-2.6 where it is now causing trouble. Some players will have a similar history, others may not.
I don't know how this happened, though; I think 2.5's Qt always defaulted to a theme called "Fusion". I don't think the theme name "Freeciv" has ever been used for Qt (unlike the other GUIs).
A few people have seen this symptom, and I don't know their settings migration history, but mine is relatively controlled: I started out with a clean slate, started 2.5 client(s) and saved their default options (from late on S2_5, I think), then started 2.6 clients which migrated their options from the 2.5 rc-file.
Updated by Jacob Nevins over 3 years ago
- Related to Bug #782349: Qt-client: assertion '((void *)0) != theme_name && theme_name[0] != '\0'' failed. added
Updated by Jacob Nevins over 3 years ago
- Related to Bug #850290: Qt client will only load new themes from the same directory as its startup theme added
Updated by Marko Lindqvist over 3 years ago
While I don't get those exact symptoms (I guess it depends on what the defaults in the system happen to be), UI think it's any (or at least many) theme names derived from 2.5.
By creating fresh 2.5 client settings file I got "Default", and that migrated to 2.6 still as "Default", and I don't think S2_6 has any handling for such a theme name (it should, but that's another matter). Similarly if one sets theme "Fusion" in 2.5, it gets verbatim to 2.6 and is not supported.
Updated by Marko Lindqvist over 3 years ago
- File 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch added
- Status changed from New to In Progress
- Sprint/Milestone set to 2.6.2
Does this patch (S2_6 version) help?
Updated by Marko Lindqvist over 3 years ago
Marko Lindqvist wrote:
Does this patch (S2_6 version) help?
It's not commit candidate. Instead Bug #853140 should be fixed first, and then this fix should be adjusted accordingly.
Updated by Marko Lindqvist over 3 years ago
- Related to Bug #853140: qtg_gui_clear_theme() doesn't fallback all the way to FC_QT_DEFAULT_THEME_NAME added
Updated by Marko Lindqvist over 3 years ago
- File 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch 0010-Qt-Fallback-to-FC_QT_DEFAULT_THEME_NAME.patch added
- Status changed from In Progress to Resolved
Patch that depends on Bug #853140, for all branches.
Updated by Marko Lindqvist over 3 years ago
- Blocks Task #853141: Release 2.6.2 added
Updated by Marko Lindqvist over 3 years ago
It was Bug #782349, by the way.
Earlier qtg_gui_load_theme() was called directly and if it failed to find the theme, it called qtg_gui_clear_theme() for setting default theme.
After the change load_theme() is being called, and it doesn't even call qtg_gui_load_theme() unless it knows that there is a theme (and where it is).
Updated by Marko Lindqvist over 3 years ago
So is the proposed patch considered
a) Completely wrong - do not commit
b) Sufficient temporary solution - commit for now, leave ticket open for a proper fix
c) Sufficient solution - commit & close ticket
Updated by Marko Lindqvist about 3 years ago
- Status changed from Resolved to Closed
- Assignee set to Marko Lindqvist
.
