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

Component: Multiselect - Provide removetokenicon hiding in case option is disabled (reopen) #15624

Open
qzai opened this issue May 20, 2024 · 0 comments · May be fixed by #15631
Open

Component: Multiselect - Provide removetokenicon hiding in case option is disabled (reopen) #15624

qzai opened this issue May 20, 2024 · 0 comments · May be fixed by #15631
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible

Comments

@qzai
Copy link

qzai commented May 20, 2024

Describe the bug

Hello,

Reopening request regarding multiselect chips removal issue:

#13065

I have case when my option is disabled, but it's selected by default. Using display="chip" causes that I can remove "disabled" option from selected list with cross. I see two ways of resolving this:

  1. Provide selectedItem context into a "removetokenicon" template
  2. Remove deleting cross of disabled tokens by default

Take a look into stackblitz where the issue can be seen.

Environment

https://stackblitz.com/edit/5r7du4?file=src%2Fapp%2Fmulti-select-chips-demo.ts,src%2Fapp%2Fmulti-select-chips-demo.html,src%2Fapp%2Fimports.ts

Reproducer

https://stackblitz.com/edit/5r7du4?file=src%2Fapp%2Fmulti-select-chips-demo.ts,src%2Fapp%2Fmulti-select-chips-demo.html,src%2Fapp%2Fimports.ts

Angular version

17.3.7

PrimeNG version

17.17.0

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

Browser(s)

No response

Steps to reproduce the behavior

Remove of the first element from selected area is possible.

Expected behavior

No response

@qzai qzai added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant