New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 752422 link

Starred by 2 users

Issue metadata

Status: Archived
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

MD notification header does not show alpha channel properly when ripple animation running

Project Member Reported by tetsui@chromium.org, Aug 4 2017

Issue description

Chrome Version: ToT

What steps will reproduce the problem?
(1) --enabled-new-style-notification
(2) Click context header to expand the notification.

What is the expected result?
Context title should be shown correctly during animation.

What happens instead?
See screenshot.
 
Screenshot 2017-08-04 at 5.25.22 PM.png
135 KB View Download
Cc: tetsui@chromium.org omrilio@chromium.org
 Issue 752802  has been merged into this issue.
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Aug 14 2017

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

commit c89b9af4d20d1072faeded95108dba642a33394f
Author: Tetsui Ohkubo <tetsui@chromium.org>
Date: Mon Aug 14 04:12:59 2017

Fix header text corruption in MD notification.

MD notifiction header label did not show properly when ripple animation
was running. According to InkDropContainerView description, this ugly
text rendering happens when paining labels to an opaque canvas with
subpixel rendering. This can be avoided by using InkDropContainerView.

TEST=manual
BUG= 752422 

Change-Id: I21e861e345de6039c142c028386701ebb7875068
Reviewed-on: https://chromium-review.googlesource.com/603033
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#494009}
[modify] https://crrev.com/c89b9af4d20d1072faeded95108dba642a33394f/ui/message_center/views/notification_header_view.cc
[modify] https://crrev.com/c89b9af4d20d1072faeded95108dba642a33394f/ui/message_center/views/notification_header_view.h

Comment 4 by tetsui@chromium.org, Aug 14 2017

Status: Fixed (was: Started)

Comment 5 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment