-
Notifications
You must be signed in to change notification settings - Fork 459
[dev-v5] Enhance TextInput component #4450
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ame change in v5 - Add Color, Search and Number to the TextInputType enum - Add example for using the TextInputTypes
|
✅ All tests passed successfully Details on your Workflow / Core Tests page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR enhances the TextInput component by renaming the property from TextFieldType to TextInputType to align with v5 naming conventions, and adds support for three new input types (Color, Search, and Number). The changes include proper documentation indicating that implementation relies on browser support and may vary between browsers.
Key Changes
- Property renamed from
TextFieldTypetoTextInputTypefor consistency with the component name - Added Color, Search, and Number types to the
TextInputTypeenum - Added comprehensive example demonstrating all input types with browser support note in documentation
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
src/Core/Enums/TextInputType.cs |
Added three new enum values (Color, Search, Number) with appropriate descriptions |
src/Core/Components/TextInput/FluentTextInput.razor.cs |
Renamed property from TextFieldType to TextInputType and added browser support note to XML documentation |
src/Core/Components/TextInput/FluentTextInput.razor |
Updated property reference from TextFieldType to TextInputType |
src/Core/Components/DataGrid/Columns/ColumnResizeOptions.razor |
Removed redundant TextFieldType property specification |
examples/Demo/FluentUI.Demo.Client/Documentation/Components/TextInput/FluentTextInput.md |
Added documentation section for TextInput types with browser support note |
examples/Demo/FluentUI.Demo.Client/Documentation/Components/TextInput/Examples/TextInputTypes.razor |
New example file demonstrating all TextInputType options |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
...s/Demo/FluentUI.Demo.Client/Documentation/Components/TextInput/Examples/TextInputTypes.razor
Show resolved
Hide resolved
...s/Demo/FluentUI.Demo.Client/Documentation/Components/TextInput/Examples/TextInputTypes.razor
Outdated
Show resolved
Hide resolved
Summary - Unit Tests Code CoverageSummary
CoverageMicrosoft.FluentUI.AspNetCore.Components - 98.7%
|
This PR makes the following changes related to the TextInput component: