#3904 closed defect (fixed)
[PATCH] Mauryans Yudhpot ship garrison number limit number doesn't match unit description
Reported by: | bugmenot | Owned by: | elexis |
---|---|---|---|
Priority: | Should Have | Milestone: | Alpha 21 |
Component: | UI & Simulation | Keywords: | patch |
Cc: | Patch: |
Attachments (6)
Change History (14)
by , 8 years ago
comment:2 by , 8 years ago
It comes from r17882. I checked other templates which could have been affected.
comment:3 by , 8 years ago
Keywords: | patch review added; Unit removed |
---|---|
Summary: | Mauryans Yudhpot ship garrison number limit number doesn't match unit description → [PATCH] Mauryans Yudhpot ship garrison number limit number doesn't match unit description |
comment:4 by , 8 years ago
Keywords: | review removed |
---|---|
Milestone: | Backlog → Alpha 21 |
- To fix the fundamentally broken mechanism of hardcoding numbers in tooltips, see #3972.
- We should ask a native speaker if the "Garrison improves the strength" should be "Garrisoning improves the strength"
- Alpha 20 showed that people are not aware of the fact that the number of units that shoot arrows is limited to 10. Since the tooltip says 13 arrows are shot at most, they will often garrison 13 units. While at it, the tooltip should clarify that while staying as short as possible.
- The siege tower shoots only one arrow for every second unit. This should either be stated in the tooltip or be reverted to 1 arrow/unit and use the limit patch from r17882.
by , 8 years ago
Attachment: | tooltip.diff added |
---|
comment:6 by , 8 years ago
Keywords: | review added |
---|
Attached patch should solve that:
- add a garrison and an arrow tooltip (so it only remains some stuff in the structure tree to do to fix #3816). They are displayed in construction panel (for cc, towers, fortress...) and in training panel (for siege tower and ships ie units with a buildingAI).
- btw fix #4061
- remove all (i hope) related numbers in templates
by , 8 years ago
Attachment: | tooltip.4.diff added |
---|
by , 8 years ago
Attachment: | tooltip.5.diff added |
---|
comment:8 by , 8 years ago
Keywords: | review removed |
---|
Thanks, that tooltip was overdue, those strings had hurt.
Note:
See TracTickets
for help on using tickets.
The mauryan trirem have a specific GarrisonHolder. I don't know if it's intended.