#6687 closed defect (fixed)

Tooltip trickle food rate for garrisoned animals

Reported by: Langbart Owned by: Grapjas
Priority: If Time Permits Milestone: Alpha 27
Component: UI & Simulation Keywords:
Cc: Patch: Phab:D4896

Description

description

  • [27244] allowed to garrison domestic animals into the corral to get a resource trickle of food.

problem

The trickle rate per animal (cow, goat, ...) is unclear.

possible solutions

Idea NO.1

Idea NO.2

  • write the trickle rate in the tooltip of the animal template

nitpick

Information about the animals garrisoning in the corral and the free supply of food is described twice.

Attachments (8)

idea_1.png (88.7 KB ) - added by Langbart 15 months ago.
idea_2.png (92.7 KB ) - added by Langbart 15 months ago.
corral.png (107.9 KB ) - added by Langbart 15 months ago.
food_trickle.png (96.0 KB ) - added by Langbart 15 months ago.
ungarrisoned_details.png (78.2 KB ) - added by Grapjas 15 months ago.
animal_details.png (62.4 KB ) - added by Grapjas 15 months ago.
garrisoned_details.png (30.5 KB ) - added by Grapjas 15 months ago.
new_tooltip.png (42.0 KB ) - added by Grapjas 15 months ago.

Download all attachments as: .zip

Change History (15)

by Langbart, 15 months ago

Attachment: idea_1.png added

by Langbart, 15 months ago

Attachment: idea_2.png added

by Langbart, 15 months ago

Attachment: corral.png added

comment:1 by wowgetoffyourcellphone, 15 months ago

Owner: set to wowgetoffyourcellphone
Resolution: fixed
Status: newclosed

In 27443:

Simulation | Tooltip trickle food rate for garrisoned animals

Fixes #6687

comment:2 by Langbart, 15 months ago

Reopening

  • correct values
    cattle 2s
    pig 3s
    sheep 4s
    goat 5s
    
  • not all animals that can technically be garrisoned are listed in the description
    • chicken/ peacock can be garrisoned in a corral if assigned to a player by a map maker (player colored)
chicken 4s
peacock 4s
  • the text from the corral can be easily overlooked when the values in the template change or when a new herd domestic animal is being added

.js solution required would be better


@wowgetoffyourcellphone the standard way is to upload a patch and get approval to avoid erroneous commits.

Last edited 15 months ago by Langbart (previous) (diff)

comment:3 by Langbart, 15 months ago

Resolution: fixed
Status: closedreopened

comment:4 by Langbart, 15 months ago

The minimum to close this ticket is to change the description of the aura

Example (the exact wording is left to the patch maker)

  • binaries/data/mods/public/simulation/data/auras/structures/corral_garrison.json

    a b  
    44    "modifications": [
    55        { "value": "ResourceTrickle/Rates/food", "add": 1 }
    66    ],
    7     "auraDescription" : "Garrison domesticated animals within the Corral to enable a slow trickle of food per animal. Pigs, Sheep, and Goats: 1 food/4 seconds. Cattle: 1 food/2 seconds.",
     7    "auraDescription" : "Garrison domesticated animals within the Corral to enable a slow trickle of food per animal. Cattle: 1 food/2 seconds, Pigs every 3s, Sheep every 4s and Goats every 5s.",
    88    "auraName": "Ranching and Corralling"
    99}

Idea NO1 would be preferred patch

by Langbart, 15 months ago

Attachment: food_trickle.png added

comment:5 by Grapjas, 15 months ago

I've been busy too ⚙ Diff View (wildfiregames.com)

Aiming to simplify the bonus handling it via 2 auras and clearing unnecessary text. Along with a hoverable tooltip next to "detailedinfo" (which can be used by anything thats adds resourceTrickle to a building). It should probably have a dedicated icon for this but just used the pop one for proof of concept.

with garrison in coral:

Last edited 15 months ago by Grapjas (previous) (diff)

by Grapjas, 15 months ago

Attachment: ungarrisoned_details.png added

by Grapjas, 15 months ago

Attachment: animal_details.png added

by Grapjas, 15 months ago

Attachment: garrisoned_details.png added

by Grapjas, 15 months ago

Attachment: new_tooltip.png added

comment:6 by Grapjas, 15 months ago

Owner: changed from wowgetoffyourcellphone to Grapjas
Patch: Phab:D4896
Status: reopenednew

comment:7 by Freagarach, 15 months ago

Resolution: fixed
Status: newclosed

In 27487:

Improving corral tooltips/info.

Reduces unnecessary text.
The corral defines the interval and the animal auras define the amount.

Patch by: @Grapjas
Differential revision: https://code.wildfiregames.com/D4896
Comments by: @Langbart, @Stan
Fixes #6687

Note: See TracTickets for help on using tickets.