MD-refresh: cr-toast paper button background color issues |
||||
Issue descriptionThere's some weirdness going on with cr-toast background, due to the recent change that we made to paper-button style to default hairline buttons to have a white background, see crbug.com/856918 .
,
Jun 27 2018
Issue 856918 has been merged into this issue.
,
Jun 27 2018
> I think we should revert the paper-button's default background to transparent, and just make it white for the manage user dialog, to reduce the chance of an unexpected breakage on dark background somewhere else. +1.
,
Jun 28 2018
,
Jun 28 2018
For now, I'm going to take the suggestion of not setting the bg-color for the default paper-button since it's a change with a larger potential impact versus changing the user-manager paper-button bg-color to white. For cr-toast, I will still set the bg-color to transparent since it is more explicit this way. Also scottchen@ noticed the bg-color needs to be fixed for the hover state as well.
,
Jun 29 2018
This is fixed in https://chromium-review.googlesource.com/c/chromium/src/+/1115354
,
Jun 29 2018
Issue 857507 has been merged into this issue. |
||||
►
Sign in to add a comment |
||||
Comment 1 by scottchen@chromium.org
, Jun 27 2018Owner: aee@chromium.org
Status: Assigned (was: Untriaged)