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

Issue 641866 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Dec 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression: Accessibility overlay is seen changed and misplaced at OOBE screen

Project Member Reported by sandeepk...@techmahindra.com, Aug 29 2016

Issue description

Version:  55.0.2842.0/8751.0.0_Dev Paine, Spring
OS: Chrome

What steps will reproduce the problem?
(1)Recover OS via USB>> go to accessibility settings on bottom left of the OOBE screen>> and observe the overlay

Expected: The UI of the overlay should not be changed and misplaced (It should be as shown in Actual screenshot)
 
Actual: Instead the UI of the overlay has been changed and misplaced (Refer screenshot)

This is a Regression issue as issue is not seen in M54: 54.0.2806.0/8639.0.0 dev-channel Spring

Issue is not applicable to Linux and Windows

 
Expected_OOBE.JPG
79.8 KB View Download
Actual_OOBE.JPG
96.0 KB View Download
Status: Untriaged (was: Unconfirmed)
Able to reproduce the issue on Chrome OS Paine device Version: 55.0.2842.0/8751.0.0 Dev.
Owner: alemate@chromium.org
Status: Assigned (was: Untriaged)
alemate@ did something change from M54 to M55 here?
Regardless, this should be fixed by new OOBE in M56, right?
Any update on this? This is marked as a 55 stable blocker, and we are nearing 55 stable. 
Status: Started (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Nov 18 2016

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

commit a713652fa5868b8d3d5dae03856f8e785b5fcff7
Author: alemate <alemate@chromium.org>
Date: Fri Nov 18 03:26:20 2016

ChromeOS OOBE: use old-style bubble for OOBE Accessibility popup.

"Classic" ChromeOS OOBE requires old-style bubble to display Accessibility
popup. This change adds special flag to revert bubble style to old design
for "Classic" OOBE.

BUG= 641866 

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

[modify] https://crrev.com/a713652fa5868b8d3d5dae03856f8e785b5fcff7/ui/login/bubble.css
[modify] https://crrev.com/a713652fa5868b8d3d5dae03856f8e785b5fcff7/ui/login/bubble.js

Labels: Merge-Request-55
Status: Fixed (was: Started)

Comment 8 by dimu@chromium.org, Nov 19 2016

Labels: -Merge-Request-55 Merge-Approved-55 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M55 (branch: 2883)
Project Member

Comment 9 by bugdroid1@chromium.org, Nov 19 2016

Labels: -merge-approved-55 merge-merged-2883
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/65137bf12da85a750f34be8e9ebfc21ab78ec7f4

commit 65137bf12da85a750f34be8e9ebfc21ab78ec7f4
Author: Alexander Alekseev <alemate@chromium.org>
Date: Sat Nov 19 08:27:20 2016

ChromeOS OOBE: use old-style bubble for OOBE Accessibility popup.

"Classic" ChromeOS OOBE requires old-style bubble to display Accessibility
popup. This change adds special flag to revert bubble style to old design
for "Classic" OOBE.

BUG= 641866 

Review-Url: https://codereview.chromium.org/2513583002
Cr-Commit-Position: refs/heads/master@{#433060}
(cherry picked from commit a713652fa5868b8d3d5dae03856f8e785b5fcff7)

Review URL: https://codereview.chromium.org/2514853003 .

Cr-Commit-Position: refs/branch-heads/2883@{#622}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/65137bf12da85a750f34be8e9ebfc21ab78ec7f4/ui/login/bubble.css
[modify] https://crrev.com/65137bf12da85a750f34be8e9ebfc21ab78ec7f4/ui/login/bubble.js

Labels: Merge-Request-56

Comment 11 by dimu@chromium.org, Nov 20 2016

Labels: -Merge-Request-56 Merge-Approved-56
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Project Member

Comment 12 by bugdroid1@chromium.org, Nov 21 2016

Labels: -merge-approved-56 merge-merged-2924
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e8e59ddb87e3326da268c2f1737141c43bae43d5

commit e8e59ddb87e3326da268c2f1737141c43bae43d5
Author: Alexander Alekseev <alemate@chromium.org>
Date: Mon Nov 21 10:09:02 2016

ChromeOS OOBE: use old-style bubble for OOBE Accessibility popup.

"Classic" ChromeOS OOBE requires old-style bubble to display Accessibility
popup. This change adds special flag to revert bubble style to old design
for "Classic" OOBE.

BUG= 641866 

Review-Url: https://codereview.chromium.org/2513583002
Cr-Commit-Position: refs/heads/master@{#433060}
(cherry picked from commit a713652fa5868b8d3d5dae03856f8e785b5fcff7)

Review URL: https://codereview.chromium.org/2521543002 .

Cr-Commit-Position: refs/branch-heads/2924@{#23}
Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059}

[modify] https://crrev.com/e8e59ddb87e3326da268c2f1737141c43bae43d5/ui/login/bubble.css
[modify] https://crrev.com/e8e59ddb87e3326da268c2f1737141c43bae43d5/ui/login/bubble.js

Status: Assigned (was: Fixed)
Accessibility drop-down is displayed below the Accessibility button instead of above. 

Attached screenshot. 

Tested on ChromeOS 8872.67.0, 55.0.2883.82. Reopening bug
IMG_20161206_151449.jpg
3.0 MB View Download
Cc: abodenha@chromium.org
Status: WontFix (was: Assigned)
I think it's a regression (if it is) from M53 to M54.
R53-8530.96.0 keeps the bubble above "Accessibility" link, but R54-8743.87.0 - below the link.


This is because "connect" dialog became shorter: 490px vs 453px.
I don't think it is important since we had it in M54.

Sign in to add a comment