Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

fix(text-field): Update outline idle border color to match design guidance #4768

Merged
merged 6 commits into from
May 30, 2019

Conversation

williamernest
Copy link
Contributor

Updates the select and text field outline idle border color to match spec.

fixes #4767

@@ -49,9 +48,11 @@ $mdc-select-disabled-ink-color: rgba(mdc-theme-prop-value(on-surface), .37) !def
$mdc-select-disabled-fill-color: mix(mdc-theme-prop-value(on-surface), mdc-theme-prop-value(surface), 2%) !default;
$mdc-select-disabled-dropdown-opacity: .37 !default;

$mdc-select-outlined-idle-border: rgba(mdc-theme-prop-value(on-surface), .38) !default;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are these design approved? Sometimes the spec is not up to date.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, just talked with design about them.

@williamernest
Copy link
Contributor Author

I'll be accepting the following screenshot diffs with this PR due to CBT upgrading their FF version from 65 to 66.
https://storage.googleapis.com/mdc-web-screenshot-tests/travis/2019/05/29/18_58_34_059/report/report.html

@mdc-web-bot
Copy link
Collaborator

🤖 Beep boop!

Screenshot test report 🚦

127 screenshots changed from master on commit 1118dc6:

Details

127 Changed:

@mdc-web-bot
Copy link
Collaborator

🤖 Beep boop!

Screenshot test report 🚦

127 screenshots changed from master on commit 4d02104:

Details

127 Changed:

@williamernest williamernest merged commit 7fedeaf into master May 30, 2019
@williamernest williamernest deleted the fix/textfield/update-border-color branch May 30, 2019 17:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[MDCTextField] Outlined variant idle border color should be 38% opacity
4 participants