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

Wrong color for validation errors #3973

Closed
calebegg opened this issue Jul 31, 2015 · 0 comments
Closed

Wrong color for validation errors #3973

calebegg opened this issue Jul 31, 2015 · 0 comments
Assignees
@EladBezalel
Labels
g3: reported The issue was reported by an internal or external product team. pr: merge ready This PR is ready for a caretaker to review ui: CSS
Milestone
1.0-rc7

Comments

@calebegg
Copy link
Member

Validation errors are #F44336 (on e.g. https://material.angularjs.org/HEAD/#/demo/material.components.select). The spec says they should be #DD2C00: http://www.google.com/design/spec/patterns/errors.html#errors-user-input-errors
@ThomasBurleson ThomasBurleson modified the milestone: Backlog Sep 1, 2015
@ThomasBurleson ThomasBurleson modified the milestones: 0.12.0, Backlog Sep 4, 2015
@jelbourn jelbourn added the g3: reported The issue was reported by an internal or external product team. label Sep 14, 2015
@ThomasBurleson ThomasBurleson modified the milestones: 1.0-rc1, 1.0-rc2, 1.0-rc3 Oct 27, 2015
@ThomasBurleson ThomasBurleson modified the milestones: 1.0-rc3, 1.0-rc4 Nov 7, 2015
@ThomasBurleson ThomasBurleson modified the milestones: 1.0-rc4, 1.0-rc5 Nov 16, 2015
@robertmesserle robertmesserle modified the milestones: 1.0-rc5, 1.0-rc8 Nov 23, 2015
@ThomasBurleson ThomasBurleson modified the milestones: 1.0-rc8, 1.0-rc7 Nov 25, 2015
EladBezalel added a commit that referenced this issue Nov 30, 2015
@EladBezalel
style(core): change warn color from red to deep-orange as spec
4a697fd 
In #3973 OP addressed us that the color of the input error should be `#DD2C00` which is `A700` from the deep-orange palette so i changed all `warn-500` (the old value from the red palette) to `warn-A700` and change the default warn palette to deep-orange.

fixes #3973
@EladBezalel EladBezalel added the pr: merge ready This PR is ready for a caretaker to review label Nov 30, 2015
@EladBezalel EladBezalel mentioned this issue Nov 30, 2015
Closed
@EladBezalel EladBezalel mentioned this issue Feb 6, 2016
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@EladBezalel EladBezalel

Labels
g3: reported The issue was reported by an internal or external product team. pr: merge ready This PR is ready for a caretaker to review ui: CSS
Projects
None yet
Milestone
1.0-rc7
Development

Successfully merging a pull request may close this issue.

style(core): change warn color from red to deep-orange as spec angular/material
6 participants
@topherfangio @ThomasBurleson @robertmesserle @calebegg @jelbourn @EladBezalel