Skip to content

fix(ui5-multi-input): expand tokenizer on focus in#13191

Merged
ndeshev merged 1 commit intomainfrom
mi-expand
Mar 6, 2026
Merged

fix(ui5-multi-input): expand tokenizer on focus in#13191
ndeshev merged 1 commit intomainfrom
mi-expand

Conversation

@ndeshev
Copy link
Contributor

@ndeshev ndeshev commented Mar 4, 2026

fixes: #13179

@ui5-webcomponents-bot
Copy link
Collaborator

ui5-webcomponents-bot commented Mar 4, 2026

🧹 Preview deployment cleaned up: https://pr-13191--ui5-webcomponents.netlify.app

@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview March 4, 2026 13:57 Inactive
Copy link
Contributor

@Vonahz Vonahz left a comment

Choose a reason for hiding this comment

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

I noticed that when Multiinput is in readonly state tokenizer will expand when try to click the input (multiinput gets focus). Before it was expanded only if tokenizer is focused.
Not sure if this is desired. Couldn't find any specifics in the spec.

@ndeshev ndeshev merged commit cd993af into main Mar 6, 2026
21 of 22 checks passed
@ndeshev ndeshev deleted the mi-expand branch March 6, 2026 13:53
@ui5-webcomponents-bot ui5-webcomponents-bot temporarily deployed to preview March 6, 2026 13:53 Inactive
@ndeshev
Copy link
Contributor Author

ndeshev commented Mar 9, 2026

I noticed that when Multiinput is in readonly state tokenizer will expand when try to click the input (multiinput gets focus). Before it was expanded only if tokenizer is focused. Not sure if this is desired. Couldn't find any specifics in the spec.

Yes, the Tokenizer should expand on input focus.

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.

[MultiInput]: Tokenizer does not expand on Focus

4 participants