wowgetoffyourcellphone Posted March 6, 2017 Report Share Posted March 6, 2017 Hi guy. I add text to the resources name, so that the player can get a little tooltip when hover over the icons in the top left. Can someone extend the resource code to allow the resource jsons to have a tooltip line like the technologies have? The reason for this extension is so that the gui at the bottom does not get screwed up. See, the editing I did made it work for the top ribbon, but screw up the bottom gui. Current modded json: { "code": "food", "name": "Food - Harvest from Animals, Berry Bushes, Fish, or Farms", "order": 1, "subtypes": { "fish": "Fish", "fruit": "Fruit", "grain": "Grain", "meat": "Meat" }, "truePrice": 100, "aiAnalysisInfluenceGroup": "ignore" } Desired json possibility: { "code": "food", "name": "Food", "order": 1, "subtypes": { "fish": "Fish", "fruit": "Fruit", "grain": "Grain", "meat": "Meat" }, "tooltip": "Harvest from Animals, Berry Bushes, Fish, or Farms.", "truePrice": 100, "aiAnalysisInfluenceGroup": "ignore" } 2 Quote Link to comment Share on other sites More sharing options...
Lion.Kanzen Posted March 6, 2017 Report Share Posted March 6, 2017 I love those colors. 1 Quote Link to comment Share on other sites More sharing options...
elexis Posted March 6, 2017 Report Share Posted March 6, 2017 5 hours ago, wowgetoffyourcellphone said: Hi guy. I hope your missing an s there 5 hours ago, wowgetoffyourcellphone said: Can someone extend the resource code to allow the resource jsons to have a tooltip line like the technologies have? That should require about 2 or 3 lines of code changed, maybe you want to look into it? Should this tooltip also be shown in other places? Maybe in the captions of the resources tab in the summary screen? In the tooltip of the resource count caption of the selection details? The carrying resources tooltip can't show it as we can't conceptually show tooltips of tooltips (because moving the mouse cursor hides the tooltip). Sometime we should be able to select starting resoures for each type individually, we could show it there then too. If the answer to the above question is no, then I can still conditionally accept your request. The condition being that you create a JustusAvramenko or wowgetoffyourcellphone account on http://code.wildfiregames.com, test the proposed fix to your request with vanilla 0ad and delenda est for functionality, try to look at the code whether it remotely makes sense and doesn't look ugly and click on accept. We have the review requirement so it needs the attention of two people: one team- or non-team member confirming that the patch is sensible and one team-member taking the full responsability of the diff by committing it. 2 Quote Link to comment Share on other sites More sharing options...
elexis Posted March 7, 2017 Report Share Posted March 7, 2017 resourceTooltips.diff Give strings 5 Quote Link to comment Share on other sites More sharing options...
wowgetoffyourcellphone Posted March 7, 2017 Author Report Share Posted March 7, 2017 7 Quote Link to comment Share on other sites More sharing options...
wowgetoffyourcellphone Posted March 7, 2017 Author Report Share Posted March 7, 2017 (edited) But for real though: Food: Harvest from Animals, Berry Bushes, Fish, or Farms Wood: Lumber from Trees Stone: Mine from Rocks or Stone quarries Metal: Mine from Metal Mines Edited March 7, 2017 by wowgetoffyourcellphone Quote Link to comment Share on other sites More sharing options...
wowgetoffyourcellphone Posted March 7, 2017 Author Report Share Posted March 7, 2017 22 hours ago, Lion.Kanzen said: I love those colors. They make the game more vibrant. Not sure why there is resistance against this. Quote Link to comment Share on other sites More sharing options...
elexis Posted March 7, 2017 Report Share Posted March 7, 2017 1 hour ago, wowgetoffyourcellphone said: Food: Harvest from Animals, Berry Bushes, Fish, or Farms Wood: Lumber from Trees Stone: Mine from Rocks or Stone quarries Metal: Mine from Metal Mines Not sure if they will be accepted, they seem a bit trivial. Also belive common nouns are more intuitive than proper nouns here. @Hannibal_Barca is into strings, other than that I don't know whom to ask who might has ability, interest and slighty more than zero time to spend. The code should support descriptions anyway though as mods might benefit from descriptions, so created a revision proposal on https://code.wildfiregames.com/D199. The offer above still stands. Quote Link to comment Share on other sites More sharing options...
elexis Posted March 16, 2017 Report Share Posted March 16, 2017 Day 10 of 500? of noone caring about reviews of this patch. Creating an account and doing a quick test will be faster, even if it means overcoming imaginary boundaries 2 Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.