Skip to content
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

custom status breaks status icons in sidebar #4921

Closed
doofus-01 opened this issue May 25, 2020 · 5 comments
Closed

custom status breaks status icons in sidebar #4921

doofus-01 opened this issue May 25, 2020 · 5 comments
Labels
Bug Issues involving unexpected behavior. Lua API Issues with the Lua engine and API. UI User interface issues, including both back-end and front-end issues.

Comments

@doofus-01
Copy link
Member

doofus-01 commented May 25, 2020

Game and System Information

  • What version of the game are you running?
    1.15.3+dev
  • Where did you download it from? (Steam, Mac App Store, our website, built from source, etc.)
    source
  • If you're using a custom build, what compiler did you use, and what commit did you build at?
    gcc 04a0fcc
  • What OS are you running?
    Linux

Describe the bug

The status icons, such as the little green skull for poison, are not displayed in the sidebar. The icons are displayed outside of the campaign, so it's probably not an engine bug.

To Reproduce

Steps to reproduce the behavior:

Try to find entangled/poisoned unit in test scenario

@doofus-01 doofus-01 added Bug Issues involving unexpected behavior. Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign UI User interface issues, including both back-end and front-end issues. labels May 25, 2020
@Pentarctagon
Copy link
Member

When you say that the icons are displayed outside of the campaign, does that mean they display correctly in single player scenarios?

@doofus-01
Copy link
Member Author

Yes, a default era MP game for example works fine.

Commenting out the call to theme.lua in the campaign's _main.cfg makes the poison status work again, so I guess that needs to be updated in some way.

@doofus-01
Copy link
Member Author

The test scenario seems to have the same problem, and it also uses wesnoth.interface.game_display.unit_status()
to show a custom status icon. I guess it's the LuaAPI that's bugged, though I don't know enough to pin it down any further.

@doofus-01 doofus-01 added Lua API Issues with the Lua engine and API. and removed Campaign (any) Deprecated tag, replaced with separate tags for each mainline campaign labels May 25, 2020
@doofus-01 doofus-01 changed the title UtBS - status icons not displaying custom status breaks status icons in sidebar May 25, 2020
@gfgtdf
Copy link
Contributor

gfgtdf commented May 25, 2020

the only related recent commit i see is the renaming of wesnoth.theme_items to wensoth.interface.game_display.

@CelticMinstrel maybe you could fix this?

@CelticMinstrel
Copy link
Member

So, does the custom status show but not the built-in one?

As far as I know, theme_items and game_display should be the same Lua object; if not, maybe that's the issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues involving unexpected behavior. Lua API Issues with the Lua engine and API. UI User interface issues, including both back-end and front-end issues.
Projects
None yet
Development

No branches or pull requests

4 participants