Opened 9 years ago

Last modified 3 years ago

#3213 closed defect

[PATCH] Structure Tree - units made from other units are not displayed — at Version 2

Reported by: elexis Owned by: s0600204
Priority: Should Have Milestone: Alpha 20
Component: UI & Simulation Keywords: patch
Cc: Patch:

Description (last modified by s0600204)

Currently the structure tree does not show units that can be produced from other units. For example the mauryan hero elephant can do maiden archers, which are basically the same as scythian archers. Almost noone knew about this unit... The tooltip also doesn't say anything about this ability. Another example is the athenian tireme which can produce sword champions and archers. Here the tooltip is also missing! To solve the problem, those units which can produce other units should be displayed along with the other buildings. The building list will be bigger than the screensize, it should be scrollable then. So (1) display those units (2) fix the tooltips and (3) make the list scrollable.

Change History (4)

comment:1 by elexis, 9 years ago

(1) see #3212 (2) Also the boats need a tooltip that states that the number of arrows fired increases with the number of units garrissoned, and that one can lookup the number of arrows fired in the tooltip of the boat. (3) see #3038

by s0600204, 9 years ago

Attachment: unitTrainingUnits.patch added

Patch to place units that train other units on same level as structures.

comment:2 by s0600204, 9 years ago

Description: modified (diff)
Keywords: review patch added
Milestone: BacklogAlpha 19
Owner: set to s0600204
Status: newassigned
Summary: Structure Tree - units made from other units are not displayed[PATCH] Structure Tree - units made from other units are not displayed

I suppose a secondary tree, available in a separate tab, is a possibility. Until then, the attached patch places units that train other units alongside structures.

(2) Everything that appears in a tooltip is lifted straight from the entity templates, and are run through the tooltips.js file in globalscripts. (3) That is relying on someone who knows how the GUI system works (such as leper) to find time to code that. It is also outside the purview of this ticket.

by elexis, 9 years ago

Attachment: screenshot_persian.png added

Thanks for the patch, works great! I didn't know that one of the persian heroes can train champions, and that their ship can produce sword cav.

Note: See TracTickets for help on using tickets.