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

Panelmenu: Blinking "focus" on first menu item #13940

Closed
lmormol opened this issue Oct 23, 2023 · 5 comments
Closed

Panelmenu: Blinking "focus" on first menu item #13940

lmormol opened this issue Oct 23, 2023 · 5 comments
Labels
Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible

Comments

@lmormol
Copy link

lmormol commented Oct 23, 2023

Describe the bug

There is an issue with blinking focus after clicking in any menu item. For a moment .p-focus is set on first menu item.

Here you can see incorrect behaviour:

https://drive.google.com/file/d/1TLmJ_U1S4va7NYzEbAJRiklLwNHmO3ah/view?usp=sharing

Environment

Mac os Apple M2
Sonoma 14

Reproducer

No response

Angular version

16.2.3

PrimeNG version

16.5.1

Build / Runtime

Angular CLI App

Language

TypeScript

Node version (for AoT issues node --version)

18.17.1

Browser(s)

Latests: Chrome, Opera, Safari

Steps to reproduce the behavior

  1. Go to Basic panel menu.
  2. Expand "Edit" group menu item.
  3. Start clicking on Left / Center / Right or Justify sub-items.
  4. Observe that for a moment focus is incorrectly set on a first menu item in this case "Left".

Expected behavior

"Focus" should be set on menu item which was clicked.

@lmormol lmormol added the Status: Needs Triage Issue will be reviewed by Core Team and a relevant label will be added as soon as possible label Oct 23, 2023
@lmormol lmormol changed the title Panelmenu: Title Panelmenu: Blinking "focus" on first menu item Oct 23, 2023
@jermylucas
Copy link
Contributor

Thanks for pointing this out. It's not a great visual behavior...

@lmormol
Copy link
Author

lmormol commented Nov 29, 2023

No big deal, thanks for fixing the issues!

Please have a look also at this one in a spare moment.

Have a nice day!

@jermylucas
Copy link
Contributor

jermylucas commented Nov 29, 2023

@cetincakiroglu This is fixed along with issue #14236

@cetincakiroglu
Copy link
Contributor

Closing the issue since it's already fixed.

@alejandrobiker
Copy link

The error continued, was it resolved?

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

No branches or pull requests

4 participants