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

Change: Improve China Overlord upgrade command button and cameo icon placements #1896

Merged
merged 2 commits into from
May 27, 2023

Conversation

xezon
Copy link
Collaborator

@xezon xezon commented May 1, 2023

This change improves China Overlord upgrade command button and cameo icon placements.

Advantages

  • The Overlord upgrade cameo icon positions and order now matches the Helix, Emperor and the patched upgrade button order.
  • The Overlord upgrade command button positions and order now matches Helix button positions and order.
  • The Overlord will keep the disabled upgrade buttons after acquiring the Bunker upgrade.
  • The Overlord and Emperor can now build the Gattling Cannon upgrade in a group selection.
  • Helix buttons and cameo icons are unchanged

Disadvantages

  • The Overlord upgrade button placement and order has changed.
  • The Overlord upgrade button placement looks a bit oddly placed unless the Bunker upgrade is built.

Upgrade buttons

Object Upgrade left Upgrade mid Upgrade right
Original Emperor Gattling Cannon
Original Overlord Battle Bunker Gattling Cannon Propaganda Tower
Original Helix Battle Bunker Propaganda Tower Gattling Cannon
Original Infantry Helix Battle Bunker
Patched Overlord (this) Battle Bunker Propaganda Tower Gattling Cannon
Patched Emperor (this) Gattling Cannon

Upgrade cameo icons

Object Cameo left Cameo mid Cameo right
Original Emperor Subliminal Messaging Propaganda Tower Gattling Cannon
Original Overlord Propaganda Tower Gattling Cannon Battle Bunker
Original Helix Battle Bunker Propaganda Tower Gattling Cannon
Original Infantry Helix Battle Bunker
Patched Overlord (this) Battle Bunker Propaganda Tower Gattling Cannon

Patched

shot_20230501_155848_4

shot_20230501_155826_3

@xezon xezon added Enhancement Is new feature or request Minor Severity: Minor < Major < Critical < Blocker Optional Is optional content China Affects China faction GUI For graphical user interface labels May 1, 2023
@xezon
Copy link
Collaborator Author

xezon commented May 1, 2023

I think this is good solution as it does not require touching Helix upgrade icons and cameo icons to streamline it.

@MTKing4
Copy link
Collaborator

MTKing4 commented May 1, 2023

This increases functionality, so it shouldn't be optional

@xezon
Copy link
Collaborator Author

xezon commented May 1, 2023

If we change core buttons, then we confuse legacy players. So this must be optional.

@MTKing4
Copy link
Collaborator

MTKing4 commented May 1, 2023

Missing out on a functionality has bigger impact than a small confusion, besides it won't be confusing at all, we didn't change the buttons order, it would be very clear, i don't think anyone would disagree if they knew why we did it

We've agreed in the past that only subjective and art/non gameplay improvements should be optional, it's not ok that optional players will have an advantage over core ones with changes like this

@xezon
Copy link
Collaborator Author

xezon commented May 1, 2023

Is not clear to me what you are on about.

If player likes legacy buttons, then he uses core patch buttons.

If player likes optimized buttons, then he uses optional patch buttons.

There is no downside, no discrimination. Players have the freedom to choose.

@MTKing4
Copy link
Collaborator

MTKing4 commented May 1, 2023

It's as simple as this, if Players choose core patch, they will miss out on being able to Multi purchase Upgrades when having Bunkerlord Among the group Selection

This is no different than the streamlining of evacuate buttons

@xezon
Copy link
Collaborator Author

xezon commented May 1, 2023

If Players choose core patch, they will miss out on being able to Multi purchase Upgrades when having Bunkerlord Among the group Selection.

Yes. It is their choice. What is the problem? There is a change log. All changes are transparent.

@MTKing4
Copy link
Collaborator

MTKing4 commented May 1, 2023

It's not much of a choice then, players are then forced to use optional just for increased functionality.

Question, is Evacuate streamlining optional?

@xezon
Copy link
Collaborator Author

xezon commented May 1, 2023

Not yet, but I am considering making Evacuate button changes mostly optional, so that legacy players do not complain about changed button positions. The main goal of core button layout is to avoid too much discomfort when transitioning from legacy game to new game, which is a very reasonable goal, because adoption comfort is very important.

@MTKing4
Copy link
Collaborator

MTKing4 commented May 1, 2023

I don't like that route, there will be a big discrepancy between core and optional and then it core players either switch to optional bundle or be at a huge disadvantage, i think that would upset the players far more than a few button order changes as they'll feel they have no real choice there.

I don't know about the others but I'm fine with sacrificing trivial muscle Memory for increased functionality, what I'm not fine with is changing them with no functionality gain, like the new Dozer buildings order, and that can be optional, but this can not

@xezon
Copy link
Collaborator Author

xezon commented May 1, 2023

I checked evacuation button changes and I think there needs to be nothing optional, apart from Helix button change.

@MTKing4
Copy link
Collaborator

MTKing4 commented May 1, 2023

Yes and nor should this

@xezon
Copy link
Collaborator Author

xezon commented May 2, 2023

Exile

I think it is fine

kABUSE

It makes absolutely no sense to make something that directly affects (and improves) gameplay optional and I have a hard time to follow the reasoning. I don't think the case "OMG the overlord keeps his upgrade command buttons visible even after you purchased an upgrade, I UNINSTALL THE PATCH NOW" exists. The order in which the command buttons are displayed is even more negligible. I personally could not for the love of god tell in which order the command buttons are currently displayed. I really have no idea why somebody would consider making this optional.

@kABUSE1
Copy link

kABUSE1 commented May 2, 2023

The most problematic part about this are custom scripted maps.

Sometimes scripters remove command buttons from units like dozers and put a different command button into the same slot position (by fixed numeric index). If now you move buttons around, this might mess up with the intended logic of the maps by putting custom command buttons above existing command buttons.

It's all good if some maps need to be re-worked for the patch. But it's impossible to accustom for the fact that people will have different command set versions by individual preference.

@xezon
Copy link
Collaborator Author

xezon commented May 2, 2023

Perhaps can just set all buttons for a given command set in a custom map?

Or reference a new command set in the object?

@kABUSE1
Copy link

kABUSE1 commented May 3, 2023

Speaking entirely on scripting level:
The "fix" would be to remove all command buttons that are potentially in a different (custom) order, then assign different command buttons in these slots in fixed order.

You cannot assign command-sets on script-level, you can only modify default command-sets per type. With scripts you have exactly two options:

  1. Remove specific command button from unit-type specific default command set, regardless of which slot it is in
  2. Assign specific command button in slot X of unit type's default command set (numeric slot 01 - 18)

@xezon
Copy link
Collaborator Author

xezon commented May 3, 2023

Essentially this would already be a problem, because users can already play with custom Command Sets, because game does not CRC checks it and game clients are compatible with different Command Sets.

So the robust way would be to remove all Command Buttons in Script, and then assign a clean set of Command Buttons always to ensure the Command Set looks as expected. Is that feasible?

@kABUSE1
Copy link

kABUSE1 commented May 3, 2023

Of course. This would be the way I do it. But the average mapper would not have this in mind when working on command buttons.

My point being, I would not make matters worse. Making this optional would make matters worse.

Right now custom command button orders in ZH 1.04 are probably insanely rare. We are talking below 1% rare. But if you offer this as an optional feature it would become a common problem.

@xezon
Copy link
Collaborator Author

xezon commented May 23, 2023

Judging by comments and interactions

  • 3 persons want this in Core
  • 2 persons are fine with either Core or Optional
  • 1 persons against Core

@xezon
Copy link
Collaborator Author

xezon commented May 23, 2023

We can also add Bunker and Propaganda Buttons to Emperor. These will be disabled, but can then be used in group selection with other Overlords:

shot_20230523_192810_1

@MTKing4
Copy link
Collaborator

MTKing4 commented May 24, 2023

I don't know about that last part, it could cause confusion seeing a bunker upgrade on an emperor giving the implication that it has one installed, the way i see it is emperor and overlord are two different units so they don't really need to be multi-upgraded.

That doesn't mean I'm necessarily against this change.

@xezon
Copy link
Collaborator Author

xezon commented May 24, 2023

Yes it is not ideal.

@kABUSE1
Copy link

kABUSE1 commented May 24, 2023

Yes it is not ideal.

You can move the disabled bunker and propa buttons to slot 17 and 18 for emporor overlords. This way they are never visible for human players.

@xezon
Copy link
Collaborator Author

xezon commented May 24, 2023

You can move the disabled bunker and propa buttons to slot 17 and 18 for emporor overlords. This way they are never visible for human players.

Buttons must take same slot to work in group selection.

@kABUSE1
Copy link

kABUSE1 commented May 24, 2023

You can move the disabled bunker and propa buttons to slot 17 and 18 for emporor overlords. This way they are never visible for human players.

Buttons must take same slot to work in group selection.

In that case both variants are not ideal.

I'd say the use-case of a player owning emperors and overlords at the same time AND having the need to equip overlords with something other than gattlings is so rare that it's acceptable that he has to do some manual group selection.

In such a case I would expect the player to only want gat upgrade, as his emperors already have propaganda, which should suffice for the mixture. And seriously, at that point, who would bother with bunker upgrades :D

@xezon xezon force-pushed the fix-overlord-commandset-2 branch from 2fbeaf8 to 30433fb Compare May 24, 2023 18:37
@xezon xezon removed the Optional Is optional content label May 24, 2023
@xezon
Copy link
Collaborator Author

xezon commented May 24, 2023

Changes promoted to core. Opening text updated. Please confirm satisfaction.

@xezon xezon force-pushed the fix-overlord-commandset-2 branch from 30433fb to ad90778 Compare May 24, 2023 18:46
@MTKing4
Copy link
Collaborator

MTKing4 commented May 24, 2023

Satisfied.

@xezon xezon force-pushed the fix-overlord-commandset-2 branch from ad90778 to 94ae09f Compare May 27, 2023 11:29
@xezon xezon force-pushed the fix-overlord-commandset-2 branch from 94ae09f to b226619 Compare May 27, 2023 11:31
@xezon
Copy link
Collaborator Author

xezon commented May 27, 2023

Documentation added and updated. Looks good.

@xezon xezon merged commit 6b432be into main May 27, 2023
@xezon xezon deleted the fix-overlord-commandset-2 branch May 27, 2023 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
China Affects China faction Enhancement Is new feature or request GUI For graphical user interface Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants