-
-
Notifications
You must be signed in to change notification settings - Fork 991
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add hooks into the GUI2 theming system #8825
Comments
Is that different from https://wiki.wesnoth.org/InterfaceActionsWML#.5Bchange_theme.5D? |
Not even slightly related. That's the in-game theme (ThemeWML), while this is GUI2. |
Would be good to have a list of specifications here, that is, the things that people would like to be customizable. @CelticMinstrel Can this be related to porting ThemeWML to GUI2? |
I wouldn't say it's related.
Though this could be a possibility, it might be worth limiting custom themes to be used only in-game (ie, dialogs that are shown during the course of a scenario). Otherwise, someone could install a broken theme that renders the game completely unplayable. Another thing: if I recall correctly, I think switching the GUI2 theme requires restarting Wesnoth at the moment, which is obviously a problem if we want to be able to use custom themes in scenarios. |
I was thinking about option to use custom
I do partly agree. I'm not developer but I thought it could be done using cpp equivalent of |
I really don’t think this is worth it for anything beyond the in-game UI. Besides the amount of work required to support it, it doesn’t really make sense to allow people to skin the whole UI. |
Though I kind of agree on this, it does make sense to allow people to skin such dialogs as the story screen or the narration dialog.
The in-game UI is (probably) a completely separate situation, as it must be built to tie into the reports system.
There are probably ways to break the game even then, but we'd maybe have a blacklist of dialogs you're not allowed to touch. For example, no customization of the "do you want to quit" dialog is allowed, and reskinning the menus is also not permitted. Could also do the reverse, a whitelist of dialogs you are allowed to touch. |
GUI2 has a built-in theming system that can in principle be used to reskin the entire game, but at present there's a number of obstacles to actually using it.
First of all, custom themes can only be added by editing the game data right now. There should be some way of loading themes from addons.
Secondly, while there is a preference to change the preferred theme (I'm not sure if it's an advanced preference or a hidden preference), there's no way for a scenario to request switching to a particular theme.
There's likely other issues as well that make the system unusable for anything beyond making a mobile theme for the iOS/Android versions.
The text was updated successfully, but these errors were encountered: