Skip to content

MudBaseInput: Make GetInputType protected#12351

Merged
ScarletKuro merged 2 commits intoMudBlazor:devfrom
ScarletKuro:input_protected
Dec 25, 2025
Merged

MudBaseInput: Make GetInputType protected#12351
ScarletKuro merged 2 commits intoMudBlazor:devfrom
ScarletKuro:input_protected

Conversation

@ScarletKuro
Copy link
Member

This makes sense to align with the other APIs. It’s also something that individual components might want to override, so access for third-party components is necessary for this API.

Checklist:

  • I've read the contribution guidelines
  • My code follows the style of this project
  • I've added or updated relevant unit tests

@mudbot mudbot bot added the enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library label Dec 25, 2025
@mckaragoz
Copy link
Member

https://github.com/MudBlazor/MudBlazor/blob/dev/src/MudBlazor/Base/MudFormComponent.cs#L72
Could we set SubscribeToParentForm to protected also? We may need to read value on some places.

@ScarletKuro
Copy link
Member Author

https://github.com/MudBlazor/MudBlazor/blob/dev/src/MudBlazor/Base/MudFormComponent.cs#L72 Could we set SubscribeToParentForm to protected also? We may need to read value on some places.

Made SubscribeToParentForm protected. This is API is whatever anyway, it's something I want to remove in 10.x when reworking validation and make it unnecessary.

@ScarletKuro ScarletKuro merged commit e6ef37f into MudBlazor:dev Dec 25, 2025
3 of 4 checks passed
@ScarletKuro ScarletKuro deleted the input_protected branch December 25, 2025 15:28
This was referenced Feb 20, 2026
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

enhancement Adds a new feature or enhances existing functionality (not fixing a defect) in the main library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants