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

Groups UI menu for group management #2828

Merged
merged 26 commits into from
Aug 29, 2023
Merged

Conversation

jwhite510
Copy link
Contributor

@jwhite510 jwhite510 commented Sep 21, 2022

Please give me some feedback on this.

Adds a new UI menu that displays all of the unit groups (1 - 10), right clicking one of the buttons assigns the current unit selection to the group, left clicking one of the buttons selects that group. It's always visible at the bottom of the screen. When the research / manufacture / build menu shows at the bottom of the screen, the group selection menu is moved above it.

The group number and also the number of units in that group is displayed over the button for each group, and it shows the question mark when there are no units assigned to that group.

Screenshot_2022-09-21_07-47-27
Screenshot_2022-09-21_07-47-48

@maxsupermanhd
Copy link
Member

maxsupermanhd commented Sep 22, 2022

UI for such things is great but here is my vision for it:
image
* not to scale
* more icons describing group damage / unit type omitted

Also it must be configurable, at least be able to turn it on/off and move around. Good setting would be to group similar units inside one group so it is not repeating same unit multiple times but just draw a number near them
Ability to switch on/off healthbars, ranks and so on is very much welcome

@KJeff01
Copy link
Member

KJeff01 commented Sep 24, 2022

A cool idea. I also agree it should be configurable with a option menu flag that toggles it on/off for user preference.

@jwhite510
Copy link
Contributor Author

It's now configurable in the options menu and the position is fixed under the build / manufacture / research menu.

Screenshot_2022-10-04_08-46-00
Screenshot_2022-10-04_08-46-49
Screenshot_2022-10-04_08-47-38

@maxsupermanhd
Copy link
Member

I really hate it sticking out on the bottom like this

@past-due
Copy link
Member

past-due commented Oct 4, 2022

I really hate it sticking out on the bottom like this

Hence the option to disable it.

Nice stuff, @jwhite510! 👍

@jwhite510
Copy link
Contributor Author

thanks! @past-due

@KJeff01 KJeff01 added this to the 4.4.0 milestone Oct 17, 2022
@past-due past-due merged commit 33e84fc into Warzone2100:master Aug 29, 2023
37 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants