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

Recent Alpha breaks embedded HamburgerMenu #3525

Closed
xagon0 opened this issue Jun 6, 2019 · 2 comments
Closed

Recent Alpha breaks embedded HamburgerMenu #3525

xagon0 opened this issue Jun 6, 2019 · 2 comments
Assignees
Labels
Milestone

Comments

@xagon0
Copy link

xagon0 commented Jun 6, 2019

As of Alpha 0312, a HamburgerMenu in a UserControl displaying a HamburgerMenu with sub views, no longer functions correctly. It appears the index of the control is drawn incorrectly.

To Reproduce: Place HamburgerMenu in MainWindow, Create UserControl with HamburgerMenu, Create UserControl with details.

Point HamburgerMenu1 to HamburgerMenu2 User Control.

Selecting Option 1, then Option 1 (of Menu 2) results in the entire control disappearing.

Initial Load
image

Select Item 1 - HamburgerMenu 1
image

Select Item 2 - Hamburger Menu 2
image

On Alpha 0262: Initial Load
image

On Alpha 0262: Select Item 1 - HamburgerMenu 1
image

On Alpha 0262: Select Item 2 - Hamburger Menu 2
image

  • OS: [Win10 1803]
  • Visual Studio [e.g. 2019 16.2.0 P1]
  • .NET Framework [e.g. 4.5.2]

Example Repo at: https://github.com/xagon0/ad0042e9-4c13-4a7c-a729-8d05444714cb

@punker76 punker76 self-assigned this Jun 6, 2019
@punker76 punker76 added the Bug label Jun 6, 2019
@punker76 punker76 added this to the 2.0.0 milestone Jun 6, 2019
@punker76
Copy link
Member

punker76 commented Jun 8, 2019

@xagon0 I found the problem for this issue and will fix it soon

punker76 added a commit that referenced this issue Jun 8, 2019
Set RoutingStrategy to Direct for all routed events to enable embedded HamburgerMenu (SubMenu).
@punker76
Copy link
Member

punker76 commented Jun 8, 2019

@xagon0 this has now been fixed, and it will be released in the next version of MahApps.Metro v2.0

@punker76 punker76 closed this as completed Jun 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

2 participants