HomeWildfire Games

Make the resource/pop tooltips sortable

Description

Make the resource/pop tooltips sortable

Patch By: ffffffff
Differential Revision: https://code.wildfiregames.com/D1113

Details

Committed
bbJan 30 2018, 12:12 AM
Differential Revision
D1113: Order players by resource or population tooltip numbers
Parents
rP21070: petra cleanup
Branches
Unknown
Tags
Unknown
Build Status
Buildable 4741
Build 8216: Post-Commit BuildJenkins

Event Timeline

elexis added a subscriber: elexis.Feb 25 2018, 7:26 PM
elexis added inline comments.
/ps/trunk/binaries/data/config/default.cfg
367

Don't use abbreviations as it requires the reader to guess what they could mean.
respoptooltipsort -> resourcesTooltipOrder

/ps/trunk/binaries/data/mods/public/gui/options/options.json
133

Everything Used Title Case Before.
Resources Tooltip Order?

137

That's ordered by playerID, I guess not relevant.

/ps/trunk/binaries/data/mods/public/gui/session/session.js
1192

Wondering if the hotkey tooltip shouldn't be last, but not important.

/ps/trunk/binaries/data/mods/public/gui/session/top_panel/resources.xml
10

The duplication is other than good.

ffffffff added inline comments.
/ps/trunk/binaries/data/config/default.cfg
367

Ok

/ps/trunk/binaries/data/mods/public/gui/options/options.json
133

Ok

/ps/trunk/binaries/data/mods/public/gui/session/session.js
1192

feel ok with it

/ps/trunk/binaries/data/mods/public/gui/session/top_panel/resources.xml
10

function into session.js?

changeResPopTooltipSort()?

elexis added inline comments.Feb 27 2018, 5:59 PM
/ps/trunk/binaries/data/mods/public/gui/session/top_panel/resources.xml
10

Yep
See my comment earlier on about abbreviations. Sort -> Order. change -> toggle.