Jump to content

Multiple unit UI not appearing


Phalanx
 Share

Recommended Posts

I think this is the error you will get ...

ERROR: JavaScript error: gui/session/selection_details.js line 416
ReferenceError: costIcon is not defined
  displayMultiple/numberOfUnits.tooltip<.details<@gui/session/selection_details.js:416:8
  displayMultiple@gui/session/selection_details.js:413:24
  updateSelectionDetails@gui/session/selection_details.js:466:3
  updateGUIObjects@gui/session/session.js:783:2
  onSimulationUpdate@gui/session/session.js:726:2
  __eventhandler65 (simulationupdate)@sn simulationupdate:0:1

Link to comment
Share on other sites

Looks like according to https://github.com/0ad/0ad/commit/5c10d03826b535a5a26e1175f23e04e85beab3f2 this function was renamed to resourceIcon() + missed in few places (guess) ...

grep -R "costIcon" *
gui/session/selection_details.js:                    { "type": costIcon(res), "amount": totalCarrying[res] })).join("  "))
gui/session/selection_details.js:                    { "type": costIcon(res), "amount": totalLoot[res] })).join("  "))
gui/session/selection_panels.js:                "population": costIcon("population"),
gui/session/selection_panels.js:                costIcon(res) + data.carried[res]

resourceIcon.patch

Edited by jonbaer
Link to comment
Share on other sites

ERROR: JavaScript error: gui/session/selection_details.js line 425 ReferenceError: costIcon is not defined displayMultiple/numberOfUnits.tooltip<.details<@gui/session/selection_details.js:425:8 displayMultiple@gui/session/selection_details.js:422:24 updateSelectionDetails@gui/session/selection_details.js:466:3 updateGUIObjects@gui/session/session.js:764:2 onTick@gui/session/session.js:650:3 __eventhandler58 (tick)@sn tick:0:1

 

even after tooltip.js commit

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

 Share

×
×
  • Create New...