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

themes: Allow primary and accent colors to be the same palette. #8260

Closed
ErinCoughlan opened this issue Apr 28, 2016 · 0 comments
Closed

themes: Allow primary and accent colors to be the same palette. #8260

ErinCoughlan opened this issue Apr 28, 2016 · 0 comments
Assignees
Labels
has: Pull Request A PR has been created to address this issue ui: theme
Milestone

Comments

@ErinCoughlan
Copy link
Contributor

ErinCoughlan commented Apr 28, 2016

Currently, Material throws an warning when primary and accent colors are the same (here). This is not disallowed in the Material spec, so the warning should be removed.

The workaround is to duplicate palettes and change the name:

$mdThemingProvider.definePalette(
      'grey-accent', $mdThemingProvider.extendPalette('grey', {}));
$mdThemingProvider.definePalette(
      'grey-background', $mdThemingProvider.extendPalette('grey', {}));
@EladBezalel EladBezalel added this to the 1.1.0 milestone Apr 28, 2016
EladBezalel added a commit that referenced this issue May 26, 2016
- This is not disallowed in the Material spec, so the warning should be removed

fixes #8260
@EladBezalel EladBezalel added the has: Pull Request A PR has been created to address this issue label May 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
has: Pull Request A PR has been created to address this issue ui: theme
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants