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

MenuFlyout should have page background color in high contrast #2432

Closed
mcooley opened this issue May 12, 2020 · 1 comment · Fixed by #2446
Closed

MenuFlyout should have page background color in high contrast #2432

mcooley opened this issue May 12, 2020 · 1 comment · Fixed by #2446
Labels
area-Menus area-UIDesign UI Design, styling help wanted Issue ideal for external contributors team-Controls Issue for the Controls team

Comments

@mcooley
Copy link
Member

mcooley commented May 12, 2020

Describe the bug
In recent versions of WinUI, MenuFlyouts have a gray background in high contrast themes. They should have the page background color.

Steps to reproduce the bug
Add the following XAML to a test app:

<Button Content="Open Flyout">
  <Button.Flyout>
    <MenuFlyout>
      <MenuFlyoutItem Text="Test 1"></MenuFlyoutItem>
      <MenuFlyoutItem Text="Test 2"></MenuFlyoutItem>
    </MenuFlyout>
  </Button.Flyout>
</Button>

Expected behavior
The background of a MenuFlyout should match the user's preferred background color, like it does in Windows.UI.Xaml.

Screenshots
image

Version Info

NuGet package version:
Microsoft.UI.Xaml 2.3.200213001
Microsoft.UI.Xaml 2.4.0

Windows 10 version Saw the problem?
Insider Build (19628.mn_release.200508-1414) Yes
November 2019 Update (18363)
May 2019 Update (18362)
October 2018 Update (17763)
April 2018 Update (17134)
Fall Creators Update (16299)
Creators Update (15063)
Device form factor Saw the problem?
Desktop Yes
Mobile
Xbox
Surface Hub
IoT

Additional context
Internal bug: 26080077.

@msft-github-bot msft-github-bot added the needs-triage Issue needs to be triaged by the area owners label May 12, 2020
@ranjeshj ranjeshj added area-Menus area-UIDesign UI Design, styling team-Controls Issue for the Controls team help wanted Issue ideal for external contributors and removed needs-triage Issue needs to be triaged by the area owners labels May 12, 2020
@ranjeshj
Copy link
Contributor

This does look like a bug. We should match the behavior of Windows.UI.Xaml in this case.

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 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.

3 participants