Opened 7 years ago

Closed 6 years ago

#676 closed enhancement (fixed)

Unit description / unit name disparity

Reported by: beherith Owned by: Bluestone
Priority: minor Milestone:
Component: BAR Version:
Keywords: Cc:

Description

This is a wierd issue, because econ structures have names that are descriptions, and descriptions that are tooltips, while most mobile units just names and proper descriptions. This really muddies tooltips and stuff, but I dont have a solution in mind.

Change History (9)

comment:1 by Bluestone, 7 years ago

We could re-write them all by hand, after agreeing which field(s) will be displayed in the UI?

comment:2 by Bluestone, 7 years ago

Component: BA trunkBAR

lets fix this as part of BAR

comment:3 by beherith, 6 years ago

n2s: add tidal and wind to desc of units in unitdefs post

comment:4 by beherith, 6 years ago

waaa, unitdefs_post doesnt have Game.*

UnitDefscorwin.description=UnitDefscorwin.description + string.format(" (%i-%iE)",Game.windMin,Game.windMax)

UnitDefsarmwin.description=UnitDefsarmwin.description + string.format(" (%i-%iE)",Game.windMin,Game.windMax)

UnitDefsarmtide.description=UnitDefsarmtide.description + string.format(" (%iE)",Game.tidal)

UnitDefscortide.description=UnitDefscortide.description + string.format(" (%iE)",Game.tidal)

comment:5 by Bluestone, 6 years ago

Hmm, I was thinking that the wind/tidal production might be better set within a gadget (then it would be possible to override maps with crazy values). The perf impact of doing this vua a call in CreateUnit should be 0. I hadn't thought of it before but it would also solve this problem.

Last edited 6 years ago by Bluestone (previous) (diff)

comment:6 by beherith, 6 years ago

Im ok with maps having crazy values...

comment:7 by Bluestone, 6 years ago

Ok, I don't think there is a way to get the Game.* into unitdefs_post, though.

Last edited 6 years ago by Bluestone (previous) (diff)

comment:8 by Bluestone, 6 years ago

Owner: set to Bluestone
Status: newassigned

comment:9 by Bluestone, 6 years ago

Resolution: fixed
Status: assignedclosed

might be details to clear up but this is basically fixed

Note: See TracTickets for help on using tickets.