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

[Degradation] Context Menu Submenus now changing main menu position on menu open #7249

Closed
Mikolaytis opened this issue Dec 23, 2021 · 6 comments · Fixed by #7250
Closed

[Degradation] Context Menu Submenus now changing main menu position on menu open #7249

Mikolaytis opened this issue Dec 23, 2021 · 6 comments · Fixed by #7250
Labels

Comments

@Mikolaytis
Copy link
Contributor

Describe the bug
Context Menu Submenus now changing main menu position on menu open.
Occurs after merge of master branch into our private repo. Around a month of commits.

To Reproduce

  1. Enable Hosted in-window popups
  2. Create a context menu with a submenu.
  3. Set it to a large control
  4. open context menu.
  5. mouse over submenu
    Only works on menus that positioned on mouse. Main Menu is fine.
151221.context.menu.mp4
@Mikolaytis Mikolaytis added the bug label Dec 23, 2021
@Mikolaytis
Copy link
Contributor Author

I've tried to find a change/pr/commit that caused the issue - looks like this is #7071
But i'm not sure.

@maxkatz6
Copy link
Member

Yes, seems to be affected only for overlay popups, which are disabled by default.

@Mikolaytis
Copy link
Contributor Author

After some debugging I've found that PlacementTargetLayoutUpdated - is fired and caused position change.

@josephnarai
Copy link

Is there a timeframe for when this fix will be available via nuget?

@maxkatz6
Copy link
Member

@josephnarai it is available now with 0.10.12.

@josephnarai
Copy link

josephnarai commented Jan 26, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants