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

Chips | Cannot remove p-focus (Firefox) #12016

Closed
lemonCMS opened this issue Oct 6, 2022 · 0 comments · Fixed by #12038
Closed

Chips | Cannot remove p-focus (Firefox) #12016

lemonCMS opened this issue Oct 6, 2022 · 0 comments · Fixed by #12038
Assignees
Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Milestone

Comments

@lemonCMS
Copy link

lemonCMS commented Oct 6, 2022

Describe the bug

Firefox bug

Provided we use p-chips with max set to 5. After reaching the fifth value and add this value by pressing enter the p-focus css class does not get removed from the ul element

In Chrome and Edge Chromium it works as aspected.

Environment

Running Angular v14 and primeNG v14

Reproducer

https://stackblitz.com/edit/primeng-chips-demo-hbcyme?file=src/app/app.component.html

Angular version

14

PrimeNG version

14

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

14

Browser(s)

Firefox 105

Steps to reproduce the behavior

Add max attribute to the chips component, keep adding values and hit enter untill max is reached.

In FF the class p-focus is still bound to the ul element while in Chrome end Edge the class has been removed.

The example provided here is running on angular v13 and primeng v13, but it has the same behavior on v14

Expected behavior

After reaching the max and hitting enter that the p-focus class has been removed from the ul element

@lemonCMS lemonCMS added the Type: Bug Issue contains a bug related to a specific component. Something about the component is not working label Oct 6, 2022
@cetincakiroglu cetincakiroglu added this to the 14.1.3 milestone Oct 12, 2022
volvachev added a commit to volvachev/primeng that referenced this issue Oct 12, 2022
@cetincakiroglu cetincakiroglu changed the title Component: Chips Chips | Cannot remove p-focus (Firefox) Nov 9, 2022
@cetincakiroglu cetincakiroglu added Type: Bug Issue contains a bug related to a specific component. Something about the component is not working and removed Type: Bug Issue contains a bug related to a specific component. Something about the component is not working labels Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Issue contains a bug related to a specific component. Something about the component is not working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants