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

Add a composite hero/leader ellipse #6258

Closed
cooljeanius opened this issue Nov 5, 2021 · 6 comments · Fixed by #8375
Closed

Add a composite hero/leader ellipse #6258

cooljeanius opened this issue Nov 5, 2021 · 6 comments · Fixed by #8375
Labels
Enhancement Issues that are requests for new features or changes to existing ones. Graphics Issues that involve the graphics engine or assets. Units Issues that involve unit definitions or their implementation in the engine.

Comments

@cooljeanius
Copy link
Contributor

If an add-on author tries to use {IS_HERO} or the ellipse misc/ellipse-hero for a unit with canrecruit=yes, they end up getting messages like error display: could not open image 'misc/ellipse-hero-leader-top.png'. (I have had this happen to me before myself.) Technically this is Wesnoth just doing what it's told, so there's no need to change any code here, but it would be useful to at least add a composite ellipse with that path for this particular case, at least IMO.

@Wedge009 Wedge009 added Enhancement Issues that are requests for new features or changes to existing ones. Graphics Issues that involve the graphics engine or assets. Units Issues that involve unit definitions or their implementation in the engine. labels Nov 5, 2021
@knyghtmare
Copy link
Member

If an add-on author tries to use {IS_HERO} or the ellipse misc/ellipse-hero for a unit with canrecruit=yes, they end up getting messages like error display: could not open image 'misc/ellipse-hero-leader-top.png'. (I have had this happen to me before myself.) Technically this is Wesnoth just doing what it's told, so there's no need to change any code here, but it would be useful to at least add a composite ellipse with that path for this particular case, at least IMO.

I have had this issue the most in my campaign for the first arc. Previously, in 1.10/1.12 series, it was possible to have both, with the hero icon being overlayed on top of the leader icon.

@cooljeanius
Copy link
Contributor Author

I have had this issue the most in my campaign for the first arc. Previously, in 1.10/1.12 series, it was possible to have both, with the hero icon being overlayed on top of the leader icon.

Hm, do you think you could bisect to find which commit changed this?

@doofus-01
Copy link
Member

I have had this issue the most in my campaign for the first arc. Previously, in 1.10/1.12 series, it was possible to have both, with the hero icon being overlayed on top of the leader icon.

I believe this is for the ellipse (side-colored things on the ground), not icons (the little floating gold/silver trinkets).

Maybe a hack, but simply copying the leader ellipse images and calling them "ellipse-hero-leader" should work. Any reason not to do that?

@cooljeanius
Copy link
Contributor Author

I have had this issue the most in my campaign for the first arc. Previously, in 1.10/1.12 series, it was possible to have both, with the hero icon being overlayed on top of the leader icon.

I believe this is for the ellipse (side-colored things on the ground), not icons (the little floating gold/silver trinkets).

Maybe a hack, but simply copying the leader ellipse images and calling them "ellipse-hero-leader" should work. Any reason not to do that?

Let's ask @irydacea

@cooljeanius
Copy link
Contributor Author

I have had this issue the most in my campaign for the first arc. Previously, in 1.10/1.12 series, it was possible to have both, with the hero icon being overlayed on top of the leader icon.

I believe this is for the ellipse (side-colored things on the ground), not icons (the little floating gold/silver trinkets).
Maybe a hack, but simply copying the leader ellipse images and calling them "ellipse-hero-leader" should work. Any reason not to do that?

Let's ask @irydacea

(it works perfectly fine when doing so as part of an add-on, btw)

@CelticMinstrel
Copy link
Member

CelticMinstrel commented Dec 18, 2021

I also think it would be perfectly acceptable to just duplicate the regular leader ellipse for this, as @doofus-01 suggested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement Issues that are requests for new features or changes to existing ones. Graphics Issues that involve the graphics engine or assets. Units Issues that involve unit definitions or their implementation in the engine.
Projects
None yet
5 participants