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

Issue 809861 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Notification border is missing when inline settings is shown.

Project Member Reported by megumihattori@google.com, Feb 7 2018

Issue description

Chrome Version: ToT (8a629223e2672e0d2aef95d1a864e22b240b99bb)
OS: Chrome OS (device name: eve)

What steps will reproduce the problem?
(1) Generate a web notification at https://tests.peter.sh/notification-generator/
(2) Click the "1" button on the bottom bar of the screen
(3) Click the gear icon on the notification

What is the expected result?
The border outside of the notification is not missing after an animation run.

What happens instead?
The border is missing.

Please use labels and text to provide additional information.


For graphics-related bugs, please copy/paste the contents of the about:gpu
page at the end of this report.

 
I've made a mistake about Chrome Version.
The correct one is ToT (a441ab2a2de6290940ca8b8f776e25aa5297fc4a).
Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 8 2018

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

commit bb18f3689f44f3ab938449e6c34ded71f06b4d72
Author: Megumi Hattori <megumihattori@google.com>
Date: Thu Feb 08 07:41:10 2018

Fix animation area when inline settings is appearing.

The animation when inline settings was appearing overlapped with the border outside of it. The border is shown only if the notification is nested.

This CL fixes the animation area.

BUG= 809861 

Change-Id: I998e9afd294fef6059de1141e7211036877c1401
Reviewed-on: https://chromium-review.googlesource.com/905965
Commit-Queue: Megumi Hattori <megumihattori@google.com>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#535329}
[modify] https://crrev.com/bb18f3689f44f3ab938449e6c34ded71f06b4d72/ui/message_center/views/message_view.h
[modify] https://crrev.com/bb18f3689f44f3ab938449e6c34ded71f06b4d72/ui/message_center/views/notification_view_md.cc

Status: Fixed (was: Started)

Sign in to add a comment