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

Add autoDisplay to MenuBar and TieredMenu #8293

Closed
nhnb opened this issue Nov 1, 2019 · 9 comments
Closed

Add autoDisplay to MenuBar and TieredMenu #8293

nhnb opened this issue Nov 1, 2019 · 9 comments
Assignees
Labels
Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add
Milestone

Comments

@nhnb
Copy link
Contributor

nhnb commented Nov 1, 2019

I'm submitting a ... (check one with "x")

[X] bug report (regression)

Current behavior

Moving the mouse above a menubar entry does not open the menu anymore.

Expected behavior

Moving the mouse above a menubar entry should popup the menu as it did in the past.

What is the motivation / use case for changing the behavior?

Restore the previous behavior because it made it easier for users to discover submenu entries.

  • PrimeNG version: master (regression from 8.0)
  • Browser:
    all

The regression seems to have been introduced intentionally as part of issue #8256 and issue #8266

@tobymurray
Copy link

tobymurray commented Dec 4, 2019

Adding on to this, approaching this from a developer perspective, it looks like it was deprecated in 8.1.0 (according to the changelog) and is now non-functional - "deprecated" is generally used to describe the process of discouraging the use of a feature, not removing it entirely. E.g. you deprecate functionality for some amount of time, then remove it some time later - this allows people making use of the work an opportunity to accommodate the deprecation. It also provides an opportunity for discussion about how to handle the deprecation. If "autodisplay" is intentionally no longer functional I personally find it misleading to describe it as "deprecated".

Secondly, #8266 contains virtually no information and no associated commits, but it does link to #8214. Going to that ticket there is nothing indicating what's going on with "autodisplay" (other than it "should behave like their PrimeReact cousins", which is not something that means anything to me).

I know it's a lot of work to introduce documentation anywhere, but it's not clear how I (as a developer looking at PrimeNG) am supposed to use the project when I come across "deprecations" like this. There is effectively no information, so the status quo appears to be unexpected functional regressions from minor releases.

@esaesa
Copy link

esaesa commented Feb 29, 2020

Any alternative to open the menu on hover?

@cagataycivici
Copy link
Member

autoDisplay is deprecated and removed in older versions.

@abashmak
Copy link

You should not simply close an issue when the ask is to provide parity with deprecated functionality. As mentioned in the request and follow up comment, there was no reason given for the deprecation, no warning period, and no alternative solution. Please address this issue properly, as it is preventing people from upgrading.

@armved
Copy link

armved commented Jun 26, 2020

+1 to @abashmak

@antsteyer
Copy link

Anybody found a workaround for this ?

@xinmoveware
Copy link

Would also like to know

@cagataycivici cagataycivici self-assigned this Oct 14, 2020
@cagataycivici cagataycivici added the Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add label Oct 14, 2020
@cagataycivici cagataycivici added this to the 10.0.4 milestone Oct 14, 2020
@cagataycivici cagataycivici reopened this Oct 14, 2020
@cagataycivici
Copy link
Member

Due to popular demand, we'll reintroduce it for 10.0.4 and 9.x.

@cagataycivici cagataycivici modified the milestones: 10.0.4, 10.1.0 Oct 19, 2020
@cagataycivici cagataycivici changed the title Menubar does not open menu on mouseover anymore (regression) Add autoDisplay to MenuBar and TieredMenu Nov 17, 2020
@MariemBADIS
Copy link

The same problem on version 9 primeNg. any one find a solution for this issues?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Issue contains an enhancement related to a specific component. Additional functionality has been add
Projects
None yet
Development

No branches or pull requests

9 participants