Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: China Overlord, Emperor command sets #1544

Closed
wants to merge 3 commits into from
Closed

Conversation

xezon
Copy link
Collaborator

@xezon xezon commented Jan 12, 2023

Merge with Rebase.

This change fixes and improves China Overlord, Emperor command sets.

TODO

  • Remake this change for optional use

Overlord

  • Upgrade buttons now have same position as Helix upgrade buttons
  • Upgrade buttons now have same order as Helix upgrade buttons, ordered left to right from least expensive upgrade to most expensive upgrade
  • Upgrade icons now have same order as Helix, Emperor upgrade icons - matching order of upgrade buttons
  • Evacuate button in Bunker Command Set now has same position as Listening Outpost, Troopcrawler evacuate button
  • Disabled upgrade buttons remain visible in Command Set - allowing use of shortcut key to research upgrade on multi unit selection

Emperor

  • Upgrade buttons now have same position as Helix upgrade buttons
  • Disabled upgrade buttons remain visible in Command Set - allowing use of shortcut key to research upgrade on multi unit selection

Original Overlord Command Set & Cameo

shot_20230112_195805_1

Patched Overlord Command Set & Cameo

shot_20230112_193923_1

Original Overlord Bunker Command Set

shot_20230112_195815_2

Patched Overlord Bunker Command Set

shot_20230112_195436_1

Original Emperor Command Set

shot_20230112_195903_3

Patched Emperor Command Set

shot_20230112_195655_2

@xezon xezon added Bug Something is not working right Enhancement Is new feature or request Controversial Is controversial Minor Severity: Minor < Major < Critical < Blocker China Affects China faction labels Jan 12, 2023
@xezon
Copy link
Collaborator Author

xezon commented Jan 12, 2023

Marked as controversial, because button positions have changed. At a glance it did not feel weird, because there is a lot of void around the Overlord upgrade buttons that make the reposition more forgiving as opposed to a more crowded command set. Changing Overlord upgrade button art later should also help improve visibility.

@Stubbjax
Copy link
Collaborator

This is a great change functionality-wise, but the movement of the buttons is a massive counterpoint that makes this change very difficult to consider.

Is there a reason the evacuate button is in a different location to the Helix's? Perhaps it should be beside the transport slots? It might be a good idea to have a look at how the evacuate slots are positioned for other units and pursue some kind of consistency.

@xezon
Copy link
Collaborator Author

xezon commented Jan 13, 2023

This is a great change functionality-wise, but the movement of the buttons is a massive counterpoint that makes this change very difficult to consider.

It is either that or legacy behaviour. Unfortunately the legacy upgrade positions collide with the garrison slots.

Is there a reason the evacuate button is in a different location to the Helix's?

Yes. Because the Helix Evacuate button is not where Outpost, Troopcrawler Evacuate button is located.

Perhaps it should be beside the transport slots?

That is legacy position.

It might be a good idea to have a look at how the evacuate slots are positioned for other units and pursue some kind of consistency.

This is what this change does. It puts Evacuate button at same position as Evacuate button of Outpost, Troopcrawler. Since Outpost, Troopcrawler are ground vehicles like the Overlord, I have choosen their Evacuate button position to take precedent over the Helix Evacuate button position.

@Stubbjax
Copy link
Collaborator

This is what this change does. It puts Evacuate button at same position as Evacuate button of Outpost, Troopcrawler. Since Outpost, Troopcrawler are ground vehicles like the Overlord, I have choosen their Evacuate button position to take precedent over the Helix Evacuate button position.

Okay, that is good. It is now shared with Battle Bus, Technical, Humvee and Ambulance.

It is strange that the Listening Outpost and Troop Crawler use a different command button for their evacuate behaviours (the icon has a vehicle instead of a building), despite the same functionality. This prevents the action from being used when those units are mixed with others. It would be good to standardise this.

@Stubbjax
Copy link
Collaborator

There is also the question of whether USA vehicle drone button positions should be taken into consideration.

@xezon
Copy link
Collaborator Author

xezon commented Jan 13, 2023

There is also the question of whether USA vehicle drone button positions should be taken into consideration.

I was thinking about this, but concluded that because USA is a different faction and China Helix already has a reasonable setup, it is sufficient to streamline Overlord button setup with Helix and other China vehicle button setups.

It is strange that the Listening Outpost and Troop Crawler use a different command button for their evacuate behaviours (the icon has a vehicle instead of a building), despite the same functionality. This prevents the action from being used when those units are mixed with others. It would be good to standardise this.

Oh interesting observation. I was already wondering why the Evacuate button was not working across different unit types, Overlord + Outpost. It gave me the impression that it is general limitation.

@MTKing4
Copy link
Collaborator

MTKing4 commented Jan 13, 2023

A few points here:

  • Does the position of the evac button really have to be the same on different units for it to function? Because if not i think it's best to keep the evac button near the slots, it's quite the standard in many units having the evac after the slots
  • while i do understand and like the upgrades position change to increase functionality, they don't have to be exactly like the helixes', i think shifting them to the right by 1 button will be optimal especially if we're putting the Evac button after the slots
  • i don't like the changed order of the upgrades, it serves no purpose and messes with player's muscle memory, again, no need to be streamlined with helix
  • the same goes of the order of icons/Cameos
  • i think for emperor, it's a different unit, so i think it's fine to put the gat upgrade at slot 1, it looks weird putting it at like in the middle of an empty command set

@xezon
Copy link
Collaborator Author

xezon commented Jan 13, 2023

Does the position of the evac button really have to be the same on different units for it to function? Because if not i think it's best to keep the evac button near the slots, it's quite the standard in many units having the evac after the slots

They need to be on same position, otherwise shortcut key does not work.

while i do understand and like the upgrades position change to increase functionality, they don't have to be exactly like the helixes', i think shifting them to the right by 1 button will be optimal especially if we're putting the Evac button after the slots

There is limitation in freedom due the 5 Bunker slots.

i don't like the changed order of the upgrades, it serves no purpose and messes with player's muscle memory, again, no need to be streamlined with helix

Fixed button order is good for new players.

the same goes of the order of icons/Cameos

Fixed icon order is good for readability.

i think for emperor, it's a different unit, so i think it's fine to put the gat upgrade at slot 1, it looks weird putting it at like in the middle of an empty command set

Perhaps. On that note: Bomb Truck upgrade positions are odd.

@MTKing4
Copy link
Collaborator

MTKing4 commented Jan 13, 2023

There is limitation in freedom due the 5 Bunker slots

You can't use the slot that's under the attack move button?

@xezon
Copy link
Collaborator Author

xezon commented Jan 13, 2023

It is strange that the Listening Outpost and Troop Crawler use a different command button for their evacuate behaviours (the icon has a vehicle instead of a building), despite the same functionality. This prevents the action from being used when those units are mixed with others. It would be good to standardise this.

@Stubbjax Stubbjax mentioned this pull request Jan 16, 2023
@xezon xezon added Optional Is optional content GUI For graphical user interface and removed Controversial Is controversial labels Apr 26, 2023
@xezon
Copy link
Collaborator Author

xezon commented Apr 26, 2023

I think it is possible to make the button position changes optional. Then it won't be controversial.

@xezon xezon added Enhancement Is new feature or request and removed Enhancement Is new feature or request labels May 1, 2023
@xezon
Copy link
Collaborator Author

xezon commented May 1, 2023

Superseeded by #1896 and #1894

@xezon xezon closed this May 1, 2023
@xezon xezon deleted the fix-overlord-commandset branch May 1, 2023 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is not working right China Affects China faction Enhancement Is new feature or request GUI For graphical user interface Minor Severity: Minor < Major < Critical < Blocker Optional Is optional content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants