Page MenuHomeWildfire Games

Plus symbol for heal ranges
ClosedPublic

Authored by Imarok on May 28 2017, 11:46 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 13, 11:16 AM
Unknown Object (File)
Fri, Sep 13, 9:57 AM
Unknown Object (File)
Wed, Sep 11, 9:50 PM
Unknown Object (File)
Tue, Sep 10, 6:49 PM
Unknown Object (File)
Sun, Sep 8, 4:23 PM
Unknown Object (File)
Thu, Sep 5, 1:47 AM
Unknown Object (File)
Fri, Aug 30, 6:10 PM
Unknown Object (File)
Sat, Aug 24, 8:58 PM
Subscribers
Restricted Owners Package

Details

Summary

This shows heal ranges(D432) as a circle out of pluses.

Test Plan

look at temples, healers and cunobelin.
textures needs to be added manually under binaries/data/mods/public/art/textures/selection/
F146433 F146432

Diff Detail

Repository
rP 0 A.D. Public Repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Owners added a subscriber: Restricted Owners Package.May 28 2017, 11:46 PM

If the auras are shown this way,maur_hero_chanakya_2.json and epic_temple_heal.json need it too (but no more than that).
What do we do to the mauryan healer who has 2 healing ranges? Leave as is (so that the config option also works as advertized)?

This is good to see, it's good to have different textures for different range visualizations (although I'm a bit uncertain if this new texture should also be used for auras that happen to heal, it may be confusing to new players).

In D571#23470, @elexis wrote:

What do we do to the mauryan healer who has 2 healing ranges? Leave as is (so that the config option also works as advertized)?

I guess this makes sense.

binaries/data/mods/public/simulation/components/Heal.js
92 ↗(On Diff #2296)

I guess a different filename should be chosen, as outline_border was used as a default, originally used exclusively for building selections. Maybe heal_overlay_range will be enough.

Executing section Default...
Executing section Source...
Executing section JS...
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/81/ for more details.

binaries/data/mods/public/simulation/components/Heal.js
92 ↗(On Diff #2296)

ok, will change that, when committing.

I really like this texture for the new heal range, thanks for that (preferably just for entities with the heal component, not for healing auras).

This revision is now accepted and ready to land.May 31 2017, 9:07 PM

Remove the plus texture from temple heal aura and cunobelin heal aura

Build is green

Updating workspaces.
Build (release)...
Build (debug)...
Running release tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Running debug tests...
Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
Checking XML files...

http://jw:8080/job/phabricator/1435/ for more details.

Executing section Default...
Executing section Source...
Executing section JS...
Executing section XML GUI...
Executing section Python...
Executing section Perl...

http://jw:8080/job/phabricator_lint/97/ for more details.

This revision was automatically updated to reflect the committed changes.