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

Issue 813008 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Notifications menu is not seen completely when device is in tablet mode

Project Member Reported by kebalaji@chromium.org, Feb 16 2018

Issue description

Chrome Version:66.0.3344.0/10408.0.0 dev channel Cyan, Minnie
OS:Chrome OS

What steps will reproduce the problem?
(1)Sign-in to user>> Add few sites in Allow section of Notifications in  Chrome://settings/Content
(2)Now Flip the device to tablet mode vertically>>Touch on Notifications>> tap on Settings icon and then on close notifications 
(3)Rotate Horizontally and touch on Notifications settings icon and observe(Refer video)

Actual: Notifications menu is not seen completely
Expected:Notifications menu should be seen completely with a scrollbar

This is a Regression issue as same is working fine on M62

NOTE: Issue is not seen on Linux and Windows
      Issue is seen on M63 and M64 stable and M65 dev also

@Yoshiki: Please confirm the issue

 
ActualNotificationsTouch.mp4
5.6 MB View Download
ExpectedNotificationTouch.mp4
3.6 MB View Download
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 23 2018

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

commit 70e505a2c6665ce1e28518422b85c3dde865ba62
Author: yoshiki iguchi <yoshiki@chromium.org>
Date: Fri Mar 23 05:44:53 2018

Close Message Center on screen rotation

This CL adds the code to close the message center when the
root window bounds is changed. When the bound is changed,
the message center may be hidden by the edge of display
partially. This CL prevents such a bad state.

Bug:  813008 
Change-Id: Ia2a2bbc319f759a8dd078a91f406e88f11235a17
Reviewed-on: https://chromium-review.googlesource.com/958049
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545373}
[modify] https://crrev.com/70e505a2c6665ce1e28518422b85c3dde865ba62/ash/system/tray/tray_background_view.cc
[modify] https://crrev.com/70e505a2c6665ce1e28518422b85c3dde865ba62/ash/system/tray/tray_background_view.h
[modify] https://crrev.com/70e505a2c6665ce1e28518422b85c3dde865ba62/ash/system/tray/tray_bubble_wrapper.cc
[modify] https://crrev.com/70e505a2c6665ce1e28518422b85c3dde865ba62/ash/system/tray/tray_bubble_wrapper.h
[modify] https://crrev.com/70e505a2c6665ce1e28518422b85c3dde865ba62/ash/system/web_notification/web_notification_tray.cc
[modify] https://crrev.com/70e505a2c6665ce1e28518422b85c3dde865ba62/ash/system/web_notification/web_notification_tray.h

Status: Fixed (was: Started)

Sign in to add a comment