Issue metadata
Sign in to add a comment
|
Regression: Misplaced notification bar overlay is seen on clicking notification icon |
||||||||||||||||||||||
Issue descriptionVersion: Started Working on 56.0.2923.0/8999.0 dev-channel Falco,Gnawty and Blaze OS: Chrome What steps will reproduce the problem? (1) Sign in to user>> go to notification bar overlay on bottom right>> click on it>> and observe Expected: Misplaced notification bar overlay should not be seen on clicking Actual: Instead misplacement is seen This is a Regression issue as no misplacement is seen in M55: 55.0.2883.54/8872.54.0 beta-channel Gnawty Issue is not applicable to Linux and Windows @Yoshiki: Please confirm the behavior
,
Nov 29 2016
Re-opening this. So at the time I wanted the panel to be aligned to the right edge, and didn't understood that the notification panel would be affected. Can we keep this behavior for panels but make an exception for the notification center only, re-aligning it as before? Sorry for the back and forth.
,
Nov 29 2016
Issue 667842 has been merged into this issue.
,
Nov 29 2016
ok.
,
Dec 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/cda46b04bb3184f699ad9dd44dcebe94fd6f5a59 commit cda46b04bb3184f699ad9dd44dcebe94fd6f5a59 Author: estade <estade@chromium.org> Date: Thu Dec 01 22:06:16 2016 Justify the message center to the right of the screen (i.e. the same place as the system menu). BUG= 666625 Review-Url: https://codereview.chromium.org/2540053002 Cr-Commit-Position: refs/heads/master@{#435733} [modify] https://crrev.com/cda46b04bb3184f699ad9dd44dcebe94fd6f5a59/ash/common/system/web_notification/web_notification_tray.cc
,
Dec 1 2016
,
Dec 2 2016
,
Dec 2 2016
,
Dec 2 2016
Your change meets the bar and is auto-approved for M56 (branch: 2924)
,
Dec 2 2016
Verified in 57.0.2939.0 canary.
,
Dec 2 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/f56dd541d1b1ee78c0a1c34910b204e440f0d293 commit f56dd541d1b1ee78c0a1c34910b204e440f0d293 Author: Terry Anderson <tdanderson@chromium.org> Date: Fri Dec 02 22:30:41 2016 Justify the message center to the right of the screen (i.e. the same place as the system menu). BUG= 666625 Review-Url: https://codereview.chromium.org/2540053002 Cr-Commit-Position: refs/heads/master@{#435733} (cherry picked from commit cda46b04bb3184f699ad9dd44dcebe94fd6f5a59) Review URL: https://codereview.chromium.org/2551443003 . Cr-Commit-Position: refs/branch-heads/2924@{#305} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/f56dd541d1b1ee78c0a1c34910b204e440f0d293/ash/common/system/web_notification/web_notification_tray.cc
,
Dec 2 2016
,
Dec 12 2016
Verified on 56.0.2924.26/9000.26.0 57.0.29460/9065.0.0
,
Dec 12 2016
Reopening as issue is still seen on the lock screen
,
Dec 19 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/8522fd00254ba2e136b89f60efb0bb6da4b7b707 commit 8522fd00254ba2e136b89f60efb0bb6da4b7b707 Author: estade <estade@chromium.org> Date: Mon Dec 19 21:43:42 2016 CrOS - Fix position of notification bubble for lock screen. In this case the alignment enum value is SHELF_ALIGNMENT_BOTTOM_LOCKED. BUG= 666625 Review-Url: https://codereview.chromium.org/2589493007 Cr-Commit-Position: refs/heads/master@{#439577} [modify] https://crrev.com/8522fd00254ba2e136b89f60efb0bb6da4b7b707/ash/common/system/web_notification/web_notification_tray.cc
,
Dec 29 2016
merge request for https://crrev.com/8522fd00254ba2e136b89f60efb0bb6da4b7b707
,
Dec 29 2016
Your change meets the bar and is auto-approved for M56 (branch: 2924)
,
Dec 29 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/6b30a3f73963c02bfc081e0178af9e3a27b5b65d commit 6b30a3f73963c02bfc081e0178af9e3a27b5b65d Author: Evan Stade <estade@chromium.org> Date: Thu Dec 29 21:43:37 2016 CrOS - Fix position of notification bubble for lock screen. In this case the alignment enum value is SHELF_ALIGNMENT_BOTTOM_LOCKED. BUG= 666625 Review-Url: https://codereview.chromium.org/2589493007 Cr-Commit-Position: refs/heads/master@{#439577} (cherry picked from commit 8522fd00254ba2e136b89f60efb0bb6da4b7b707) Review-Url: https://codereview.chromium.org/2602143002 . Cr-Commit-Position: refs/branch-heads/2924@{#633} Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059} [modify] https://crrev.com/6b30a3f73963c02bfc081e0178af9e3a27b5b65d/ash/common/system/web_notification/web_notification_tray.cc
,
Dec 29 2016
,
Jan 23 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by yoshiki@chromium.org
, Nov 29 2016Status: WontFix (was: Assigned)