Skip to content

MudTextField: Add async postfixes#12484

Merged
ScarletKuro merged 4 commits intoMudBlazor:devfrom
91378246:rename-async-mud-text-field-functions
Jan 18, 2026
Merged

MudTextField: Add async postfixes#12484
ScarletKuro merged 4 commits intoMudBlazor:devfrom
91378246:rename-async-mud-text-field-functions

Conversation

@91378246
Copy link
Contributor

@91378246 91378246 commented Jan 18, 2026

  • This PR adds the async postfix to all async functions of MudTextField
  • This PR is breaking
  • Note: SetTextAsync from MudTextField collides with SetTextAsync from MudBaseInput. The new definition of SetTextAsync in MudTextField is now public override async Task SetTextAsync(string? text) therefore overwriting the definition in MudBaseInput which should be the intended way if I am not mistaken but please take a look at it

@mudbot mudbot bot added breaking change This change will require consumer code updates enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library labels Jan 18, 2026
@danielchalmers danielchalmers requested a review from henon January 18, 2026 18:05
@danielchalmers danielchalmers removed the enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library label Jan 18, 2026
@ScarletKuro
Copy link
Member

Actually, now when I think it, I'd play it differently. I will revert last commit and merge as is. And make PR with my idea.

@ScarletKuro ScarletKuro merged commit 6552d93 into MudBlazor:dev Jan 18, 2026
5 checks passed
@91378246 91378246 deleted the rename-async-mud-text-field-functions branch January 18, 2026 18:37
This was referenced Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking change This change will require consumer code updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants