Skip to content

feat(match2): adjust bracket match widths in smash - #8127

Open
Hesketh2 wants to merge 1 commit into
mainfrom
Hesketh2-patch-2
Open

Hesketh2 wants to merge 1 commit into
mainfrom
Hesketh2-patch-2

Conversation

@Hesketh2

@Hesketh2 Hesketh2 commented Sep 20, 2026

Copy link
Copy Markdown
Collaborator

Summary

image image

This would be a good QoL to go alongside with my attempt at encouraging more Match2 usage on Smash, that is to avoid the clipping names by stretching the matchWidth noticeably, but not too much. It should cover most cases so user dont have to type matchWidth all the time

Note

190 is the value I took from MLBB info file, seems working here

@Hesketh2 Hesketh2 self-assigned this Sep 20, 2026
@Hesketh2
Hesketh2 requested review from a team as code owners September 20, 2026 15:40

@steve020607 steve020607 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Seems okay from me

@ElectricalBoy ElectricalBoy left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@ElectricalBoy ElectricalBoy changed the title feat: add adjusted matchWidth for Smash feat(match2): adjust bracket match widths in smash Sep 21, 2026
@hjpalpha

Copy link
Copy Markdown
Collaborator

not a fan

see previous discussions:

imo worth it if they finally start using match2

@Hesketh2

Copy link
Copy Markdown
Collaborator Author

not a fan

see previous discussions:

* [#7668 (comment)](https://github.com/Liquipedia/Lua-Modules/pull/7668#issuecomment-4833194755)

* https://discord.com/channels/321003439431745537/321641940879802368/1524214855823921263

we'd be fine for now with stretching the bracket, unless we can somehow convince the editor community to move back to numeric scores (at this rate the contributor are still even hanging on the point that they dont see why they should move to M2, and 1 is better. So let alone our attempt to tell them of ditching head scores vs Numbers atm. Probably best to start slo and see how it works out)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants