-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Creators Hamburger Menu #2975
Creators Hamburger Menu #2975
Conversation
I'll correct the |
That's beautiful! Good job! :) Some points about the mouse hover event based on the W10 Hamburger Menu:
But that's a design choice. Anyway, it's incredible. |
@mira r I Alex can you snag some screen grabs for me? I'll make those bindable properties so that they can be adjusted to user needs, and default them back to match the creators update |
@MurariAlex thank you! @punker76 , @thoemmi , in your opinion, should I simply follow the design established in the Creator's update (screen shots above), or should I make the MouseOver / SelectedItem colors, sidebar thickness, etc. dependency properties to allow users further customization options? I know that "software bloat" is a big concern, so I'd like some guidance from maintainers before proceeding forward. The argument for allowing customization is that this is similar to the TabItem underlines, which are all modifiable. |
@MurariAlex I just realized that my comment earlier didn't have your full tag 😆 that's what I get for posting from my phone... thanks, autocorrect... |
No problem, @amkuchta, it happens a lot to me, at least the autocorrect didn't change it to some kind of bad word as it already did to me. 😄 |
@MurariAlex I decided to stop being lazy and open Windows Defender myself 😆 I'm glad I did - I noticed that my side lines don't quite match those of the Creators update - mine are too tall, and should match the line height of the text for the selected HamburgerMenuItem, not the height of the HamburgerMenuItem itself (if that makes sense). See below - hovering over the selected item illustrates my point: I'll implement a fix to address this, as well (probably tomorrow, as it is already rather late where I am) |
Hmmm, I see. |
@MurariAlex I've already got an idea on how to implement - I'm working it now. Gonna take a moment because I'm creating some dependency properties to allow users some customization options. In a nutshell, I'm going to use the Underline control developed for the TabControl. |
@MurariAlex Ok, the latest commit:
|
Wow, looks amazing. Good job. 😄 |
Yeah, @punker76 😉 |
Does this hamburger menu have an overlay mode like the UWP version does? |
@Amrykid He Alex :-D The |
Almost forgot about this... @punker76 , any chance we see this on 1.6.0? |
@MurariAlex maybe. It's on my list for 1.6.0... |
I'm excited. Hopefully it doesn't require too much work to incorporate, @punker76 . I tried to be as thorough as possible when I put it together. |
What changed?
Describe the changes you have made to improve this project.
Closed issues.