Skip to content
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

[multiInput] close button for token in the menu of multiInput is "clickable" even in readonly mode #9862

Closed
nnaydenow opened this issue Sep 13, 2024 Discussed in #9825 · 0 comments · Fixed by #9905
Closed
Assignees
Labels
bug This issue is a bug in the code TOPIC RL

Comments

@nnaydenow
Copy link
Contributor

nnaydenow commented Sep 13, 2024

Discussed in #9825

Originally posted by jiawang1 September 5, 2024
Hi colleague:

When I set readonly to multiInput component, the close button for the token in the input field will disappear. However, the close button for the token in the dropdown will still be visible and will react to user clicks. although it does not really delete the token from the menu.

button

I have noticed the notes: "A read-only component can not be deleted or selected, but still provides visual feedback upon user interaction." So I did not report this as a bug. Just want to confirm is this the expected design? I think click close button but nothing happen will confuse the user.

Regards
Jay

Hello @SAP/ui5-webcomponents-topic-rl ,

As you can seen in the discussion the tokens inside ui5-multi-input has close button when the component is set to readonly mode. EXAMPLE

Could you please check?

Regards,
Nayden

@nnaydenow nnaydenow added bug This issue is a bug in the code TOPIC RL labels Sep 13, 2024
@nikoletavnv nikoletavnv self-assigned this Sep 16, 2024
nowakdaniel pushed a commit that referenced this issue Sep 25, 2024
…adonly (#9905)

fix(ui5-multi-input): tokens are not deletable when multi-input is readonly-tokens

Fixes #9862
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug in the code TOPIC RL
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

2 participants