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

Proposal: Get rid of uneven left/right spacing on MenuFlyout #1445

Closed
chigy opened this issue Oct 15, 2019 · 15 comments · Fixed by #1457
Closed

Proposal: Get rid of uneven left/right spacing on MenuFlyout #1445

chigy opened this issue Oct 15, 2019 · 15 comments · Fixed by #1457
Assignees
Labels
area-Menus area-UIDesign UI Design, styling bug Something isn't working help wanted Issue ideal for external contributors team-Controls Issue for the Controls team

Comments

@chigy
Copy link
Member

chigy commented Oct 15, 2019

We noticed this with #1079 which was pointing out the space on Top/Bottom which is by design.

However, during the conversion, design and I noticed the gap on left/right and further noticed that they don't seem to be even on both sides... We would like this gap to be removed and the hover state to go edge to edge.

image

@chigy chigy added the feature proposal New feature proposal label Oct 15, 2019
@shaheedmalik
Copy link

I agree. The way it currently is, the hover isn't even centered.

It should all the way across.

@jevansaks
Copy link
Member

@chigy does this need to be a feature proposal? Seems like we should just fix it as a bug?

@jevansaks jevansaks added area-Menus area-UIDesign UI Design, styling help wanted Issue ideal for external contributors bug Something isn't working and removed feature proposal New feature proposal labels Oct 15, 2019
@marcelwgn
Copy link
Collaborator

If this is a bug I would be happy to take my shot at it. 😃

@chigy
Copy link
Member Author

chigy commented Oct 15, 2019

@jevansaks , I debated if I should open a bug but technically, this is a design change request so I opened a feature proposal. No problem with that being handled as a bug.

@adrientetar
Copy link

Also, when opening a submenu, shouldn't the item text vertically align with base menu text? Seems like the padding at the top isn't accounted for currently.

@chigy
Copy link
Member Author

chigy commented Oct 18, 2019

@adrientetar , good eyes! We were about to open a request to fix that. The select visual should be aligned. Would you do me an honor if you don't mind? (Ignore the vertical circle)

image

@mdtauk
Copy link
Contributor

mdtauk commented Oct 18, 2019

@chigy, looking at the image
image

Shouldn't the sub menu be casting a shadow on the lower menu? At the moment they blend together

@adrientetar
Copy link

Would you do me an honor if you don't mind?

Huh, are you asking me to open an issue about it? See #1462

Shouldn't the sub menu be casting a shadow on the lower menu? At the moment they blend together

Yeah, I'd say go ahead and open a design bug

@chigy
Copy link
Member Author

chigy commented Oct 18, 2019

@adrientetar , thanks for the design bug!

@chigy
Copy link
Member Author

chigy commented Oct 18, 2019

Shouldn't the sub menu be casting a shadow on the lower menu? At the moment they blend together

Please feel free to open design bug. This could have been a shadow limitation but we should document it nonetheless...

@chigy
Copy link
Member Author

chigy commented Oct 18, 2019

Shouldn't the sub menu be casting a shadow on the lower menu? At the moment they blend together

Please feel free to open design bug. This could have been a shadow limitation but we should document it nonetheless...

@adrientetar , I confirmed this is a technical limitation that we cannot draw shadow on shell popups's sub-menus.

@adrientetar
Copy link

@chigy, could we have an updated design toolkit? here: https://docs.microsoft.com/en-us/windows/uwp/design/downloads/

@chigy
Copy link
Member Author

chigy commented Oct 21, 2019

@chigy, could we have an updated design toolkit? here: https://docs.microsoft.com/en-us/windows/uwp/design/downloads/

@adrientetar , we are currently working on it with update for WinUI2.2.

@mdtauk
Copy link
Contributor

mdtauk commented Oct 22, 2019

Would be good if UI Toolkits were released at the same time as a WinUI version in the future. Maybe formally tie the two together.

@chigy
Copy link
Member Author

chigy commented Oct 22, 2019

Would be good if UI Toolkits were released at the same time as a WinUI version in the future. Maybe formally tie the two together.

@mdtauk , I understand, I'd like that also. We are doing what we can.

@jevansaks jevansaks added the team-Controls Issue for the Controls team label Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Menus area-UIDesign UI Design, styling bug Something isn't working help wanted Issue ideal for external contributors team-Controls Issue for the Controls team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants