New issue
Advanced search Search tips

Issue 898278 link

Starred by 5 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 1
Cc:
Components:
EstimatedDays: ----
NextAction: 2018-11-05
OS: Mac
Pri: 2
Type: Bug



Sign in to add a comment

CSS graytext is not rendering correctly in latest Chrome in macOS 10.14 Mojave

Reported by david.ho...@gmail.com, Oct 23

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_14_0) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.67 Safari/537.36

Example URL:
https://embed.plnkr.co/IglxZCGZBAN3iA0A4Oys/

Steps to reproduce the problem:
1. Load the example site https://embed.plnkr.co/IglxZCGZBAN3iA0A4Oys/
2. Notice that the rendered style is not "graying" out properly (text is black and button is not appearing disabled)

What is the expected behavior?
Button should be restyled to appear that it is not clickable

What went wrong?
When a button gets the "disabled" attribute applied, previously a default "graytext" style was applied. Now, while the style is applied (per the "computed styles" pane) there is no obvious difference in visual appearance of an enabled and disabled button.

This worked properly in previous versions of macOS and renders correctly in Safari 12.0 but seems to also be broken in Firefox as well

Does it occur on multiple sites: Yes

Is it a problem with a plugin? No 

Did this work before? N/A 

Does this work in other browsers? No
 Firefox 63.0

Chrome version: 70.0.3538.67  Channel: stable
OS Version: OS X 10.14.0
Flash Version:
 
chrome.png
6.5 KB View Download
safari.png
6.3 KB View Download
firefox.png
6.4 KB View Download
Labels: Needs-Triage-M70
Cc: phanindra.mandapaka@chromium.org
Labels: Needs-Feedback Triaged-ET Proj-MacMojave
david.hocky@ Thanks for the issue...

Unable to reproduce the issue on reported chrome version 70.0.3538.67 using Mac 10.13.6. Attaching screen-cast for reference.
Steps: 
---------
1. Launched reported chrome 
2. Navigated the URL "https://embed.plnkr.co/IglxZCGZBAN3iA0A4Oys/"
As we are Observed that the Button is not clickable and seen same as safari

@Reporter: Request you to retry this issue with fresh profile without any extensions & apps or reset all the flags and let us know if issue still persists.

Thanks.!
898278.mp4
1.5 MB View Download
phanindra.mandapaka@chromium.org thanks for reaching out. i will give this a try. just a small clarification, this was also working for me on 10.13.x (High Sierra) it only broke after upgrading to 10.14.x (Mojave). Could you let me know if you are still unable to reproduce in 10.14?
Project Member

Comment 4 by sheriffbot@chromium.org, Oct 24

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: susan.boorgula@chromium.org
Labels: Target-72 M-72 FoundIn-71 FoundIn-72 FoundIn-70
Status: Untriaged (was: Unconfirmed)
david.hocky@ Thanks for the update.

Able to reproduce the issue on Mac OS 10.14(Mojave) on the latest Stable 70.0.3538.77 and the latest Canary 72.0.3591.0.
Issue is not reproducible on Mac OS 10.13.3, Windows 10 and Ubuntu 17.10.
Attached is the screen shot for reference.

This is a Non-Regression issue as this is observed from M-60 chrome builds.
Hence marking this as Untriaged for further updates from Dev.
Thanks..
898278-M60.png
153 KB View Download
Components: -Blink Blink>Paint
Who has a Mac with 10.14 on it? As far as I know we can't install that yet.
We Mac VMWare images at http://go/mac-litterbox.
Cc: ellyjo...@chromium.org
Labels: Needs-Feedback
NextAction: 2018-11-05
ellyjones@, does the Mac team have any idea what might have changed in Mac 10.14 to affect theme settings? From memory we draw the buttons and other theme elements with OS primitives, so it's likely they need updating in some way.
Owner: lgrey@chromium.org
Status: Assigned (was: Untriaged)
Over to lgrey@.
Looks like the system color changes introduced for dark mode broke how we interpret NSColor.disabledControlTextColor. I'm surprised we haven't seen any other breakages like that (and I imagine it might get worse when we allow actual dark mode).
Status: Started (was: Assigned)
Correction: it's the actual change to the color's value that mattered: we're ignoring alpha when translating system colors in Blink's layout theme on Mac.
Project Member

Comment 14 by bugdroid1@chromium.org, Oct 31

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/36de6d6adeb99fb05f79daab1868568f4cca367c

commit 36de6d6adeb99fb05f79daab1868568f4cca367c
Author: Leonard Grey <lgrey@chromium.org>
Date: Wed Oct 31 02:02:30 2018

Mac: Take alpha channel into account when reading system colors in Blink

Bug:  898278 
Change-Id: I24ae7ef4e8accd54244dfd09721fa376dadfba34
Reviewed-on: https://chromium-review.googlesource.com/c/1305498
Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
Commit-Queue: Leonard Grey <lgrey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604111}
[modify] https://crrev.com/36de6d6adeb99fb05f79daab1868568f4cca367c/third_party/blink/renderer/core/layout/layout_theme_mac.mm

Labels: TE-Verified-M72 TE-Verified-72.0.3598.0
Able to reproduce this issue on Mac OS 10.14 on the reported version 70.0.3538.67 and the issue is fixed on the latest M-72 build 72.0.3598.0.

On navigating to the above given URL, can observe that the Button is grey in color and is not clickable.
Attached is the screen shot for reference. 

Hence adding TE verified labels as the fix is working as intended.

Thanks..
898278-M72.png
239 KB View Download
Labels: -TE-Verified-M72 -TE-Verified-72.0.3598.0
Status: Fixed (was: Started)
The NextAction date has arrived: 2018-11-05
Thanks for the quick triage and fix to this issue! Looking forward to the GA release
Cc: lgrey@chromium.org
 Issue 911247  has been merged into this issue.

Sign in to add a comment