bugfix: Evacuate button positions are now consistent across all ground transport vehicles - #160
Conversation
e0b314c to
f982ace
Compare
Stubbjax
changed the title
Stubbjax
added
the
Committee2026
xezon
left a comment
xezon
left a comment
There was a problem hiding this comment.
The changed referenced in the Migration Project does not match this change: TheSuperHackers/GeneralsGamePatch#1545
One is moving positions, and the other is changing images.
I will unlink them. Many of Patch1's changes are not atomic so it's sometimes difficult to achieve a 1:1 match. |
|
Yes it looks like it has multiple commits in it |
|
Here is the evacuate button position reference change: TheSuperHackers/GeneralsGamePatch#165 |
| CommandSet Nuke_ChinaListeningOutpostCommandSet | ||
| 1 = Command_TransportExit | ||
| 2 = Command_TransportExit | ||
| 9 = Command_Evacuate |
There was a problem hiding this comment.
What is the rationale for not streamlining the command/icon?
There was a problem hiding this comment.
It is a separate change, and vehicles should use Command_EmptyCrawler (it shows a vehicle), structures use Command_Evacuate (it shows a building).
There was a problem hiding this comment.
Why should it be a separate change when the intended outcome is that all ground transport vehicles can use the evacuate command within the same selection?
The different icon is not important; the action is the same and it should be consistent. Command_EmptyCrawler should be obsoleted.
There was a problem hiding this comment.
Why should it be a separate change when the intended outcome is that all ground transport vehicles can use the evacuate command within the same selection?
Ok it is fair point. We can have it one change then.
The different icon is not important; the action is the same and it should be consistent. Command_EmptyCrawler should be obsoleted.
I vote for the crawler image for vehicles. It fits better. Maybe we need a vote then.
There was a problem hiding this comment.
That still does not explain how it is helpful for the user. If the action is the same, the icon should be the same unless there is a strong enough justification for otherwise.
There was a problem hiding this comment.
Just preference. I like the crawler image. It is closer to what happens: evacuating the vehicle. It is not evacuating a building.
There was a problem hiding this comment.
Personal preference should not supersede the correct logical approach. The evacuate icon represents evacuation, which is what the action does. The object being evacuated from in the icon is irrelevant.
There was a problem hiding this comment.
Who defined that? EA added 3 different evac button and it mismatches your logical approach. I understand your perspective and it is reasonable, but it is also perfectly reasonable to have a different opinion than yours.
There was a problem hiding this comment.
Did EA make a perfect game with no mistakes? They clearly had an Command_EmptyCrawler command which was likely distinct in behaviour from Command_Evacuate at some point, but the behaviour was ultimately dropped and the command was not appropriately replaced. I would also suggest a similar thing happened with the Chinook, where it would maybe have unloaded objects via ropes as depicted in the icon. It is extremely unlikely they would intentionally have had a unique Chinook unload button designed otherwise.
| 8 = Command_TransportExit | ||
| ; 9 = Command_TransportExit | ||
| ; 10 = Command_TransportExit | ||
| 9 = Command_Evacuate |
| 7 = Command_TransportExit | ||
| 8 = Command_TransportExit | ||
| 9 = Command_EmptyCrawler | ||
| 9 = Command_Evacuate |
| CommandSet ChinaListeningOutpostCommandSet | ||
| 1 = Command_TransportExit | ||
| 2 = Command_TransportExit | ||
| 9 = Command_Evacuate |
| 7 = Command_TransportExit | ||
| 8 = Command_TransportExit | ||
| 9 = Command_EmptyCrawler | ||
| 9 = Command_Evacuate |
…d transport vehicles
f982ace to
0d1ac9c
Compare
2 participants
This change streamlines the position and iconography of the Evacuate button for all ground transport vehicles. Units in a selection that share the same button position can use the hotkey for that button. This means that all units with the same Evacuate button can now be evacuated in a group. This is particularly notable while having a group selection containing both a Humvee and Ambulance.
Before (left) and after (right) layouts are compared below: