Skip to content

Conversation

Absotively
Copy link
Contributor

About the PR

  • Battlecry is now updated when the user edits it even if they don't press enter
  • Battlecry length limit is now enforced in UI; you can't enter a too-long battlecry that will be clipped
  • Battlecry window layout can no longer be broken by making the window narrower
  • When the battlecry window is opened, it is populated with the current battlecry
  • When the battlecry window is opened, the LineEdit grabs focus, and any existing battlecry is selected, so the user can immediately begin typing a new battlecry

Why / Balance

Prevent the battlecry being unchanged when it looks changed in the window. Allow the user to check the current battlecry without attacking something.

Technical details

Adjusted window xaml, added more dialog setup, modified change detection to use OnTextChanged instead of OnTextEntered.

Media

Before
image

New min width
image

Requirements

Breaking changes

  • MeleeSpeechWindow::OnBattlecryEntered is now MeleeSpeechWindow::OnBattlecryChanged and fires more often.
  • MeleeSpeechWindow::SetCurrentBattlecry will not update the displayed battlecry edit control if that control has the keyboard focus.

Changelog
🆑

  • tweak: Battlecry changes don't require pressing enter, are visible on battlecry window reopen
  • tweak: Can't enter too-long battlecry that then gets cut off

@PJBot PJBot added S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. Changes: UI Changes: Might require knowledge of UI design or code. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/S Denotes a PR that changes 10-99 lines. labels Sep 13, 2025
@iaada iaada added P3: Standard Priority: Default priority for repository items. T: UI / UX Improvement Type: UI and player facing interactive graphical interfaces A: General Interactions Area: General in-game interactions that don't relate to another area. and removed S: Untriaged Status: Indicates an item has not been triaged and doesn't have appropriate labels. labels Sep 13, 2025
@Absotively
Copy link
Contributor Author

I should mention that the changes here are similar enough to the hand labeler changes in #40318 that it might save time reviewing to look at both at once.

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

Labels

A: General Interactions Area: General in-game interactions that don't relate to another area. Changes: UI Changes: Might require knowledge of UI design or code. P3: Standard Priority: Default priority for repository items. S: Needs Review Status: Requires additional reviews before being fully accepted. Not to be replaced by S: Approved. size/S Denotes a PR that changes 10-99 lines. T: UI / UX Improvement Type: UI and player facing interactive graphical interfaces

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants