-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
feat(input): add styles for clear action to ionic theme #29337
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
The input UX requirements were updated to change all references to the Active state to just be when the input is focused. Updated the PR description to remove extra notes about what Active means. |
@amandaejohnston please update the description with the new logic for the visibility of the clear icon. |
The logic used here hasn't actually changed (it was always equivalent to the input being focused), so no updates needed on that front 👍 |
This reverts commit 3bec67f.
This reverts commit ee8fa3c.
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.
LGTM, minor nitpick that I leave up to you to add: add padding to the tests so the focus ring doesn't get cut off.
Good idea, added 👍 |
Issue number: Internal
What is the new behavior?
For the
ionic
theme: Adjusted the size of the clear button to match design specs, and updated the behavior so the button additionally only shows if the native input or any other actions within theion-input
are focused. (See code comments for details.)The actual icon used has not been modified. The ticket says to "use the material design clear icon as the reference icon for this feature," and we've already added a feature allowing the icon to be customized at the app level. Let me know if the intention was to use a different default icon with the
ionic
theme.Does this introduce a breaking change?
Other information