Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

menu: dark theme no more handled #10062

Closed

Description

@zeprone

Actual Behavior:
Hi, I just update angular-material from 1.0.8 to 1.1.1 and md-menu is no more handling dark-theme.
CodePen (or steps to reproduce the issue): *
http://codepen.io/anon/pen/KNqGpr
this codepen is the md-menu from angular-material website, configured to use dark-theme on 1.1.1

http://codepen.io/anon/pen/qqjJZZ
this one is the same one but with 1.0.9 version.

I did some test, regression occured in 1.1.0 version.

Regards,
Benoit.

Activity

  1. self-assigned this
    on Dec 15, 2016
  2. ElPsyCongro commented on Dec 23, 2016

    @ElPsyCongro

    +1

  3. clshortfuse commented on Dec 24, 2016

    @clshortfuse
    Contributor

    Related #9100

    PR #9212

    The current debate is whether to render light or dark background, if it's a regression or correction in 1.1.0

  4. zeprone commented on Jan 7, 2017

    @zeprone
    Author

    based on https://material.io/guidelines/style/color.html#color-themes
    we should have a dark background, like the dialog one in section "dark theme"

  5. clshortfuse commented on Jan 27, 2017

    @clshortfuse
    Contributor

    Closing, since duplicate of #9100

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

needs: team discussionThis issue requires a decision from the team before moving forward.ui: theme

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions