Opened 11 years ago

Last modified 5 years ago

#1774 new enhancement

[PATCH] The player doesn't know the number of his own garrisoned units at first glance

Reported by: eingousef Owned by: svott
Priority: Nice to Have Milestone: Backlog
Component: UI – In-game Keywords: patch
Cc: Patch:

Description (last modified by serveurix)

Edit: has been discussed on the forum, see : http://www.wildfiregames.com/forum/index.php?showtopic=16938

If you want to know how many of your own units(*) are garrisoned inside your own(*) siege tower, you need to click on the siege tower. If you have ten siege tower and want to know if they are all full, you need to click ten times. Two possible designs to solve this :

  • indicate the number of garrisoned units (with numbers, little dots, a bar...) when the mouse focuses on the unit/building
  • always indicate the number of garrisoned units. Maybe this requires the indication to be more aesthetic (flags, number/dots on the flag...), as it's always displayed, maybe not, as the enemies will not see it (I don't even know if enemies see the garrison flags. I think they shouldn't : it should be the surprise.).

Edit: Most games have a "display all lifebars" mode you activate/deactivate with a keyboard shortcut. Maybe the best idea is the mouse focus + a shortcut to switch to a mode where all indications about units are displayed if the player wants.

(*) or allied. I don't know if you can put units in ally buildings/siege units. Edit: It seems that #1807 made it possible, at least for buildings. If it's the case (and I think it should be the case), print the respective number of units in different colors.

PS: Same idea should apply to garrisoned units inside garrisoned units. When I click on a war ship and see four battering rams inside of it, I should know at first glance how many units are garrisoned inside each ram (but this doesn't need to be aesthetic : a simple text hover at mouse focus on each ram is enough).

Attachments (2)

0adTicket_1774.patch (1.2 KB ) - added by kingadami 11 years ago.
Initial patch ment to drive discussions
0adTicket1774_statusbars.patch (2.9 KB ) - added by kingadami 11 years ago.
The status bar implementation which only shows up for your own units

Download all attachments as: .zip

Change History (16)

comment:1 by eingousef, 11 years ago

Summary: Tha player doesn't know the number of his own garrisoned units at first glanceThe player doesn't know the number of his own garrisoned units at first glance

comment:2 by eingousef, 11 years ago

Description: modified (diff)

comment:3 by kingadami, 11 years ago

Owner: set to kingadami
Status: newassigned

I have quickly thrown together a quick implementation of this using the status bars. I will post a patch soon. I am also planning on opening up a thread in the forum to discuss this topic in detail (do we want to use a status bar or some other reporting method, should the enemy be able to see how many are garrisoned ect.)

by kingadami, 11 years ago

Attachment: 0adTicket_1774.patch added

Initial patch ment to drive discussions

comment:4 by eingousef, 11 years ago

Description: modified (diff)

by kingadami, 11 years ago

The status bar implementation which only shows up for your own units

comment:5 by kingadami, 11 years ago

Added a patch with just the status bar implementation.  The garrison status bar will only show up for your units and not another player's.  I am currently adding the ability to put a label instead of a status bar.  This is one of historic_bruno's suggestions.

comment:6 by eingousef, 11 years ago

Description: modified (diff)

comment:7 by historic_bruno, 11 years ago

Component: Core engineUI & Simulation
Keywords: patch review added; garrison siege indication removed
Summary: The player doesn't know the number of his own garrisoned units at first glance[PATCH] The player doesn't know the number of his own garrisoned units at first glance

comment:8 by sanderd17, 11 years ago

Keywords: review removed
Milestone: BacklogAlpha 15

This patch should not reuse the supply bar. Even if you can't make the icon yourself, you should copy some existing icon, but separate the code. So the only thing the artists must do is fixing the icon, and not messing with the code.

Also, it's best to remove warnings from the code.

Sorry for this late review, but as this patch didn't show up in the review queue, we've forgot about it.

comment:9 by wraitii, 10 years ago

Milestone: Alpha 15Alpha 16

comment:10 by sanderd17, 10 years ago

Milestone: Alpha 16Backlog

No progress on this ticket for a complete release, I will backlog it.

comment:11 by historic_bruno, 10 years ago

Owner: kingadami removed
Status: assignednew

comment:12 by svott, 8 years ago

Owner: set to svott

*after reading the forum post, comments and so on*

I have an idea and will try to implement it. Because I would like to see how many units are garrisoned (current/max) in the selected structures. I prefer an icon solution instead of a status bar and the icon is only visible if the structure is selected (like the status bars).

Use case: Double (or triple) click on a building and see the number of garrisoned units for each selected building

comment:13 by serveurix, 7 years ago

Description: modified (diff)

You can't really see the number of garrisoned units at first glance still. You need to select a garrisonnable unit first, or a building, and then move the mouse over the building or mechanical unit to see how much it has inside.

So when you want to embark an army in a fleet of ships, typically there's a point where some ships are already full and some next to the shore are blocking the other ships in the back. Just to know which ships you should move away from the shore you need to select a garrisonable unit or a building, move the mouse over every single one of the ships, remember which ones are full (and keep track of them if they are constantly moving, which is generally the case with ships' current behavior), and move them away one by one.

I am in favor of a hotkey to activate the garrison bubble, like there's one to activate the aura range visualization now. I am also in favor of adding a prop (like a flag) that would show to every player if a ship contain units or doesn't.

comment:14 by Imarok, 5 years ago

Component: UI & SimulationIn-game UI

Move tickets to In-game UI as UI & Simulation got some sub components.

Note: See TracTickets for help on using tickets.