Skip to content

Conversation

@KevinZK
Copy link
Collaborator

@KevinZK KevinZK commented Oct 24, 2025

Evaluate semantic message issues (particuarly with Text Input components: text field form cell, key value form cell, note form cell, title form cell) to align with Android:
1.character count
.Padding on right side
.error state color change
2.typing state Label color
3.Icon inside text box: alignment with text & behavior with expanding text box
4.error message prioritization over other semantic messages

@KevinZK KevinZK requested a review from janhuachu October 24, 2025 06:51
@KevinZK KevinZK requested a review from a team as a code owner October 24, 2025 06:51
@KevinZK KevinZK requested review from billzhou0223 and removed request for a team October 24, 2025 06:51

// sourcery: CompositeComponent, InternalComponent
// sourcery: CompositeComponent
protocol _TextInputInfoViewComponent: _InformationViewComponent, _CounterComponent {}
Copy link
Contributor

Choose a reason for hiding this comment

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

This _TextInputInfoViewComponent changed to public without header docs. Please add header docs so that the generated structs will have docs.

Copy link
Contributor

@janhuachu janhuachu left a comment

Choose a reason for hiding this comment

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

Please add header docs for _TextInputInfoViewComponent in CompositeComponentProtocols.
Also, try to add header docs for new public APIs if possible.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants