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

add ability to name groups in lobby #667

Merged
merged 4 commits into from
May 7, 2017
Merged

Conversation

commy2
Copy link
Contributor

@commy2 commy2 commented May 5, 2017

When merged this pull request will:

  • see title

They said it's impossible and it kinda is, except for this method. The only information that can be send to a client in the lobby is the role descriptions of the slots. remoteExec doesn't work and the description.ext is only available to the server machine.
By hiding our custom group name inside the role name of the first slot of the group, we can replace the default group name with anything using listbox fuckery.

This is how it looks:
http://i.imgur.com/3ACmEQt.jpg

Using "Team Leader@Team 1", the result will be a slot named "Team Leader" and the group will be named "Group 1". @ is the magic separator.

Result:
http://i.imgur.com/A0IDYHb.jpg

Since this doesn't require any information from the description.ext (slot names are send anyway to anyone in the lobby) this does work for clients.

Negatives:
All old slot names using @ will be cut off on the first @, but I doubt anyone uses this.
The slot has to be named for a group name to be chosen and it has to be manually entered.
If no respawn is used in the mission, not only the group leader, but all units have to add the group name. This is because only the first unit of the group is checked and dead units are removed from the group permanently including their slot from the lobby.

@commy2 commy2 added the Feature label May 5, 2017
@commy2 commy2 added this to the 3.4 milestone May 5, 2017
@jonpas
Copy link
Member

jonpas commented May 5, 2017

I think it would be neat if this would automatically do setGroupId on player init so it would show in-game as well. As right now, you'd have to do setGroupId and also write it here to make it the same in-game and in-lobby.

@commy2
Copy link
Contributor Author

commy2 commented May 5, 2017

@jonpas You can already change the "Callsign" attribute of the group to do this. This is broken in MP in vanilla, but also fixed in CBA (unrelated to this).
One difference between the "Role Description" of a unit and the "Callsign" of a group is, that the former exclusively affects the lobby, while the latter is used ingame as well in a bunch of places and uses the setGroupID command instead of being purely a mission.sqm entry.
I know it sucks that you basically have to enter the same information twice, but I don't want that the "Role Description" entry is used as a replacement for the "Callsign". Maybe someone wants the callsign to be different from the lobby description too, who knows.

@commy2
Copy link
Contributor Author

commy2 commented May 5, 2017

Also, as @rebelvg pointed out, you can use the same lobby name for multiple groups on the same side twice, which the Callsign attribute doesn't allow for.

@Drofseh
Copy link
Contributor

Drofseh commented May 6, 2017

All old slot names using | will be cut off on the first |, but I doubt anyone uses this.

commy and I have discussed this a little on slack, but both the communities I am part of use | in slot naming to separate platoon, squad, and team designations.

for example
https://i.gyazo.com/76bf3a84266adb1ad358d3e2a118ec5d.png
and it looks even cleaner in game than it does in this screenshot.

As such we would definitely prefer a different character be used for the splitString.

I would suggest ^, if possible, as a reminder that what follows is being moved up to the group name

for example:
1 Platoon | 2 Squad | Squad Leader^1'2
1 Platoon | 2 Squad | Alpha Team | Team Leader
1 Platoon | 2 Squad | Alpha Team | Rifleman
etc.

@jonpas
Copy link
Member

jonpas commented May 6, 2017

Semi-colon ;?

Otherwise, why not something like that: 2nd Squad{Group X}, encapsulated in {} (or something else like that except () obviously).

@commy2
Copy link
Contributor Author

commy2 commented May 6, 2017

Make \| escape it?

@Drofseh
Copy link
Contributor

Drofseh commented May 6, 2017

1 Platoon | 2 Squad | Squad Leader\|1'2
Like that?
I'd be ok with this.

@Dorbedo
Copy link
Contributor

Dorbedo commented May 7, 2017

How about using another symbol as separator? There are many left e.g.: §$%&#

@commy2
Copy link
Contributor Author

commy2 commented May 7, 2017

@

@Dorbedo
Copy link
Contributor

Dorbedo commented May 7, 2017

lgtm:
Teamlead@Group 1
Rifleman (AT)@Group 3

@commy2
Copy link
Contributor Author

commy2 commented May 7, 2017

Changed it to @.

@commy2
Copy link
Contributor Author

commy2 commented May 7, 2017

Doc: https://github.com/CBATeam/CBA_A3/wiki/Name-Groups-in-Lobby

@commy2 commy2 merged commit 2c71ff9 into master May 7, 2017
@commy2 commy2 deleted the group-names-setup-menu branch May 7, 2017 14:25
@nomisum
Copy link

nomisum commented Jun 16, 2017

you are a fucking god 🥇

@commy2 commy2 mentioned this pull request Oct 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants