Skip to content

Fix #5975#8335

Merged
melloware merged 2 commits intoprimefaces:masterfrom
kyybo:issue-5975
Oct 14, 2025
Merged

Fix #5975#8335
melloware merged 2 commits intoprimefaces:masterfrom
kyybo:issue-5975

Conversation

@kyybo
Copy link
Contributor

@kyybo kyybo commented Oct 14, 2025

Fix #5975

* Since version 10.3.0, a condition has been added to prevent submenus from activating on hover when in “popup” mode (commit 5615c89).
I don't really understand why this constraint exists. But removing it resolves issue primefaces#5975 and aligns with PrimeVue's behaviour.

* little format fix in Slider.js
@melloware
Copy link
Member

@kyybo did you look at Git Blame and see when and why that line was changed?

@kyybo
Copy link
Contributor Author

kyybo commented Oct 14, 2025

Yes, as I said in my commit message, this line appeared since the commit 5615c89
However, there is no explanation as to why the condition && !props.popup is present.

When we look at PrimeVue code, there is only the condition on the dirty property but not the popup property :
https://github.com/primefaces/primevue/blob/master/packages/primevue/src/tieredmenu/TieredMenu.vue#L306

@melloware
Copy link
Member

OK run npm run format

@melloware melloware added this to the 10.9.8 milestone Oct 14, 2025
revert from commit b435248

I ran the command "npm run format", but my version of Prettier was not up to date, my bad.
@melloware melloware merged commit 2dbbc66 into primefaces:master Oct 14, 2025
2 checks passed
@melloware melloware added the Type: Bug Issue contains a defect related to a specific component. label Oct 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Bug Issue contains a defect related to a specific component.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TieredMenu: Submenu 'popup' behavior different than PrimeVue

2 participants

Comments