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

🐞 - Pasting comma-separated values ​​into MultiSelect will delete all values #8290

Closed
2 of 9 tasks
Nastasiassss opened this issue Aug 1, 2024 · 3 comments · Fixed by #9345
Closed
2 of 9 tasks
Assignees
Labels
P1 This issue has high priority P3 This issue has low priority S3 This issue has low severity

Comments

@Nastasiassss
Copy link

Playground Link

https://taiga-ui.dev/components/multi-select#ids

Description

  1. Open https://taiga-ui.dev/components/multi-select#ids
  2. Insert into the value field: Yoda, Obi-Wan Kenobi, Han Solo

Actual behavior
Only the following is inserted into the field: Han Solo

Multi.Select.mov

Angular version

17.3.8

Taiga UI version

3.85.0

Which browsers have you used?

  • Chrome
  • Firefox
  • Safari
  • Edge

Which operating systems have you used?

  • macOS
  • Windows
  • Linux
  • iOS
  • Android
@waterplea
Copy link
Collaborator

Also when writing and typing , it would clear the value if the entered string did not match any available value.

@waterplea waterplea added contributions welcome Architecture is clear and community can help P3 This issue has low priority S3 This issue has low severity and removed state: need triage labels Sep 18, 2024
@waterplea
Copy link
Collaborator

We should match all coma separated values to possible values. And if they do not match - keep input value as is.

@vladimirpotekhin vladimirpotekhin added P1 This issue has high priority and removed contributions welcome Architecture is clear and community can help labels Oct 1, 2024
@vladimirpotekhin vladimirpotekhin self-assigned this Oct 1, 2024
@spanevin
Copy link

spanevin commented Oct 1, 2024

It looks like it uses InputTag or another similar component internally, InputTag has [separator] property, which has ',' as the default value.
So, it is possible to disable separator completely, or change it to another value.
If it's needed to keep this behavior in MultiSelect, then the separator property should be promoted to MultiSelect API and described in docs, I believe.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 This issue has high priority P3 This issue has low priority S3 This issue has low severity
Development

Successfully merging a pull request may close this issue.

4 participants