Chrome image viewer displays white around circular images even with transparency
Reported by
93m4qau...@gmail.com,
Jan 4 2018
|
|||||||
Issue descriptionUserAgent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.132 Safari/537.36 Example URL: https://chromiumbugs.appspot.com/static/images/logo.png Steps to reproduce the problem: 1. Open the chromium logo linked above. What is the expected behavior? Since the chromium logo is circular and the image does utilize transparency (as it should), the black background should cover the corners as well. What went wrong? There are boxy white corners around the chromium logo on a black background even though transparency is properly utilized in the image. 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? N/A Chrome version: 63.0.3239.132 Channel: stable OS Version: 6.1 (Windows 7, Windows Server 2008 R2) Flash Version:
,
Jan 6 2018
Correct. If the background is going to be black, then transparency should be black too.
,
Jan 6 2018
Thank you for providing more feedback. Adding requester "vamshi.kommuri@techmahindra.com" to the cc list and removing "Needs-Feedback" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jan 7 2018
,
Jan 8 2018
Thanks for the confirmation! As per the confirmation given by the reporter from comment#2 considering black border having chrome with boxy white edges as an issue, able to reproduce it on reported version 63.0.3239.132 and on the latest 65.0.3313.0 using Windows 10, Mac 10.13.1 and Ubuntu 14.04. The UI is different in M50(50.0.2661.0), as the issue/UI is seen from the introduction of M57 hence considering it as non-regression and marking it as Untriaged. Thanks!
,
Jan 10 2018
We're probably explicitly deciding to put a white background so that transparent images blend with white. I agree it looks silly here Should be easy enough to fix but does leave open the question of what exactly we would like to appear behind a transparent or partially transparent image in the image viewer like this. UI, what do you think?
,
Jan 10 2018
The background is a checkerboard pattern - it's just very faint/low contrast. (Maybe the gray should be darker...)
,
Jan 10 2018
Independently confirmed what fs@ said. I'll see if it can be made better but we're not going to change the fundamental approach.
,
Jan 10 2018
If we're going to put something other than a checkerboard pattern in the rest of the page (e.g. fill it with black as in the screenshots above), then I agree with comment 2 that the image should be "transparent to that background color". This is problematic when using Chrome like a true image editor, but I think better reflects user expectations in the common case.
,
Jan 10 2018
Personally, my opinion would be that since Chrome is simply a viewer and not an image editor, it should just display the transparency to the background color (similar to how Windows Photo Viewer displays transparency), because that's what transparency is. In my opinion, there isn't really a point in using transparency if it's going to be interpreted by a viewer as a checkerboard pattern. On the other hand, if it were an image editor (which is not the case here), I would say that a checkerboard pattern would be the most appropriate for representing transparency. But that's all just my 2 cents.
,
Jan 10 2018
I also think we're a viewer so will change it to be the same as the body background (effectively no background to the image), which has the pleasant side effect of being much more efficient for us to paint.
,
Jan 10 2018
I agree with changing it to the background color over the checkerboard pattern, since it is simply a viewer and not an image editor. Imagine transparency in favicons being displayed as a checkerboard pattern. If transparency is not displayed as "transparent" (where the color is the same as if it were not part of the image) but rather displayed as a checkerboard pattern even in viewers, then transparency is no longer true "transparency" as it was originally intended.
,
Jan 11 2018
And thanks 93m4qau783@ for engaging and pointing out room for improvement. We appreciate it.
,
Jan 11 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/eab1ec606de5921ae4e78ce5ce690076e0a37a71 commit eab1ec606de5921ae4e78ce5ce690076e0a37a71 Author: Stephen Chenney <schenney@chromium.org> Date: Thu Jan 11 17:10:44 2018 [PE] Use body background for image urls We had been displaying image urls with a very low contrast light colored checkerboard, as you might see in image editing software. There was agreement on the team to change it to use the page background color, which better matches our function as a image display platform. BUG= 799168 R=pkasting@chromium.org,fs@opera.com Change-Id: Ia676b05c80f91db9d7c5798e1938f1da6435db1b Reviewed-on: https://chromium-review.googlesource.com/860498 Commit-Queue: Stephen Chenney <schenney@chromium.org> Reviewed-by: Peter Kasting <pkasting@chromium.org> Reviewed-by: Fredrik Söderquist <fs@opera.com> Cr-Commit-Position: refs/heads/master@{#528650} [modify] https://crrev.com/eab1ec606de5921ae4e78ce5ce690076e0a37a71/third_party/WebKit/LayoutTests/images/huge-image-viewport-scale-expected.txt [modify] https://crrev.com/eab1ec606de5921ae4e78ce5ce690076e0a37a71/third_party/WebKit/Source/core/html/ImageDocument.cpp [modify] https://crrev.com/eab1ec606de5921ae4e78ce5ce690076e0a37a71/third_party/WebKit/Source/core/html/ImageDocument.h
,
Jan 11 2018
,
Jan 12 2018
Verified on Chromium build 528863 / Chromium version 65.0.3319.0 from download-chromium.appspot.com.
,
Jan 31 2018
Can you change the status to Verified?
,
Jan 31 2018
|
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by vamshi.k...@techmahindra.com
, Jan 5 2018Labels: Needs-Triage-M63 Triaged-ET Needs-Feedback
287 KB
287 KB View Download