Skip to content

Conversation

@xezon
Copy link
Contributor

@xezon xezon commented Apr 7, 2023

This change has been reverted with #1851

This change removes the Rally Point button from all command sets. It will not be missed.

Rationale

The button is obsolete. Rally Point functionality is always enabled as long as structure is selected.

Original

shot_20230407_141859_1

Patched

shot_20230407_142002_1

@xezon xezon added Design Is a matter of game design Minor Severity: Minor < Major < Critical < Blocker GUI For graphical user interface labels Apr 7, 2023
@commy2
Copy link
Collaborator

commy2 commented Apr 7, 2023

I remember someone objecting to this change when I made it in my project. I don't remember why though. As far as I can tell, this button is completely pointless.

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 7, 2023

I agree that it is pointless, but I'm not a fan of removing things that could help newcomers to the game if any, especially if we're not in dire need of that extra slot currently

@xezon
Copy link
Contributor Author

xezon commented Apr 7, 2023

In what way does it help newbies though? Clicking this button really changes nothing. The only this this button does it tell that this structure has Rally Points.

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 7, 2023

Yeah in case there's someone confused and doesn't know anything about RTS and Wondering what does the flag does, he can read the command button to understand

Basic RTS tutorial in Essence.

@commy2
Copy link
Collaborator

commy2 commented Apr 7, 2023

Set Rally Point

Tooltip:

Set the rally point for this building

Idk if that ever helped anyone.

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 7, 2023

Yeah I'm not too fussed about removing it either, just stating what the purpose of it was wether it helped anyone in the past or not.

@commy2
Copy link
Collaborator

commy2 commented Apr 7, 2023

If ingame info about rallypoints is truly needed (I don't think so personally), then it should be placed better than on a button that does absolutely nothing. I mean, this thing is more confusing right now than clearing up anything, because the button does literally nothing at all (as far as I can tell). Not even change cursor or anything.

@MTKing4
Copy link
Collaborator

MTKing4 commented Apr 7, 2023

Placed better like where?

@commy2
Copy link
Collaborator

commy2 commented Apr 8, 2023

Anywhere really. Anywhere that is not a useless button, that can be clicked, but nothing happens for some reason.

@xezon
Copy link
Contributor Author

xezon commented Apr 9, 2023

Merge this?

@xezon xezon force-pushed the remove-rallypoint-button branch from 221116b to 717431e Compare April 15, 2023 13:41
@xezon xezon force-pushed the remove-rallypoint-button branch from 717431e to 0b595aa Compare April 15, 2023 13:48
@xezon
Copy link
Contributor Author

xezon commented Apr 15, 2023

By coincidence I noticed that the button was not removed from Superweapon Command Center. Fixed. Documentation added. Looks good.

@xezon xezon merged commit 44960c6 into main Apr 15, 2023
@xezon xezon deleted the remove-rallypoint-button branch April 15, 2023 13:49
@kABUSE1
Copy link
Collaborator

kABUSE1 commented Apr 16, 2023

There is a situational difference between having the command button and not having it:

2023-04-16.14-48-36.mp4

If you have the command button you can place the rally point into a group of units via the hotkey without accidentally selecting any of them.

I only noticed after looking for a difference, I Have never done this in an actual match.

@xezon
Copy link
Contributor Author

xezon commented Apr 16, 2023

Ohh. So it does make a difference. Well then we can revert this change again :D

@kABUSE1
Copy link
Collaborator

kABUSE1 commented Apr 16, 2023

So let's do the opposite, the boss general airfield is lacking this command button by default lmao

@kABUSE1
Copy link
Collaborator

kABUSE1 commented May 31, 2023

Please also take this into consideration:
#1985

(Nevermind, current issue is already closed. I am still in the progress of understading how git works lol)

@xezon
Copy link
Contributor Author

xezon commented Jun 5, 2023

This change was merged and reverted.

@ItsWazzaP ItsWazzaP added the GameMismatchWith104Maybe Maybe affects game state towards original 1.04 label Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Design Is a matter of game design GameMismatchWith104Maybe Maybe affects game state towards original 1.04 GUI For graphical user interface Minor Severity: Minor < Major < Critical < Blocker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants