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

Issue 678558 link

Starred by 3 users

Issue metadata

Status: Verified
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug
Team-Accessibility



Sign in to add a comment

Non-Regression : Notification overlay is overlapped with Uber Tray when ChromeVox is enabled

Project Member Reported by mmanchala@chromium.org, Jan 5 2017

Issue description

Chrome Version: 56.0.2924.51/9000.50.0 Beta-channel Daisy,Candy and Peppy
OS: Chrome

What steps will reproduce the problem?
(1)Sign into User -> Click on Uber Tray and Click Accessibility
(2)Select ChromeVox(spoken feedback) option -> Now click on Uber Tray and press 'Ctrl+F5(Overview Mode Button)' for screenshot 
(3)Now observe Notification overlay overlapped with Uber Tray (please refer Video and screenshot)

Expected: Notification overlay should not be overlapped with Uber Tray when ChromeVox is enabled 
Actual: Instead Notification overlay is seen overlapped with Uber Tray  

This is Non-Regression issue seen from M-54 Build

@Yoshiki: Please confirm the Issue

 
Actual_Overlapping.jpg
321 KB View Download
Actual_Overlapping.webm
2.0 MB View Download
Expected_Space.jpg
207 KB View Download
Cc: yoshiki@chromium.org
Owner: yhanada@chromium.org
Hanada-san, could you take a look?
Status: Started (was: Assigned)
I've uploaded a fix, https://crrev.com/2625663002

Project Member

Comment 3 by bugdroid1@chromium.org, Jan 11 2017

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

commit e2867aed46b3f00f3735d310144d6ef638f8be1a
Author: yhanada <yhanada@chromium.org>
Date: Wed Jan 11 01:24:46 2017

Fix calculation of the bubble view height.

BUG= 678558 
TEST=Notification doesn't overlap with Uber Tray when ChromeVox is
     enabled.

Review-Url: https://codereview.chromium.org/2625663002
Cr-Commit-Position: refs/heads/master@{#442756}

[modify] https://crrev.com/e2867aed46b3f00f3735d310144d6ef638f8be1a/ash/common/system/tray/system_tray.cc

Status: Fixed (was: Started)
Status: Verified (was: Fixed)

Sign in to add a comment