Skip to content

New mutators + search bar#116

Merged
NicEastvillage merged 6 commits into
masterfrom
new-mutators
Jun 6, 2026
Merged

New mutators + search bar#116
NicEastvillage merged 6 commits into
masterfrom
new-mutators

Conversation

@VirxEC
Copy link
Copy Markdown
Contributor

@VirxEC VirxEC commented Jun 5, 2026

There are so many mutators now that I notice myself not knowing where the thing I want is - so I added a search bar and increased the maximum number of columns to 5 as well as the minimum number of columns to 2.

image image image

@NicEastvillage
Copy link
Copy Markdown
Contributor

I like the search functionality. That's gonna help a lot!
We could also consider grouping them under headers. In Rocket League, there are three columns: "Match", "Game", and "Ball". These groups aren't ideal, and we have many hidden mutators to consider too, but they're a starting point.

billede

I just tried to make my own groups and ended up with the following: Car/Controls, Ball, Game modes, Scoring, and Match. The mutators are actually pretty evenly spread like this. I am not quite sure what some of the new mutators do, but they sound like they go in the Game Mode-altering group.

billede

Comment thread frontend/src/components/MatchSettings/rlmutators.ts Outdated
Comment thread frontend/src/components/MatchSettings/rlmutators.ts Outdated
Copy link
Copy Markdown
Contributor

@NicEastvillage NicEastvillage left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not happy with how the modal resizes when you search. Could we make it size static?

@VirxEC
Copy link
Copy Markdown
Contributor Author

VirxEC commented Jun 6, 2026

Could we make it size static?

Err, that would probably involve hard-coding a fixed width/height... imo the resizing modal really isn't that bad, and it keeps the search results in the center of the screen and not off to a corner.

@VirxEC
Copy link
Copy Markdown
Contributor Author

VirxEC commented Jun 6, 2026

I thought it was odd at first too, but then I got used to it and I kinda like it

@NicEastvillage NicEastvillage merged commit a749e51 into master Jun 6, 2026
@VirxEC VirxEC deleted the new-mutators branch June 6, 2026 15:25
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.

2 participants