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

Issue 777830 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Regression : 'Expand/Collapse Notification' Icon is seen missing for Default Notifications

Project Member Reported by mmanchala@chromium.org, Oct 24 2017

Issue description

Chrome Version: 64.0.3246.0/10062.0.0 dev-channel Celes,Peppy and Paine
OS: Chrome

What steps will reproduce the problem?
(1)Sign in to user which has Sync Passphrase -> In sign-in screen click on Notification and observe for 'Expand/Collapse Notification' Icon at 'now' text(Please refer Video and screenshot)

Expected: 'Expand/Collapse Notification' Icon at 'now' text should be seen for Default Notifications
Actual: Instead  'Expand/Collapse Notification' Icon at 'now' text is seen missing for Default Notifications

This is Regression Issue seen as same is working fine in 64.03241.0/10038.0.0 dev-channel Daisy

@Yoshiki: Please confirm the Issue

Note : 
1.'Expand/Collapse Notification' Icon is seen on second click at 'Notifications' icon 
2. For saved or downloaded Files 'Expand/Collapse Notification' Icon is seen
 
Actual_CollapseOrExpandIcon.mp4
18.3 MB Download
Actual_CollapseOrExpandIcon.jpg
337 KB View Download
Attaching Expected video & screenshot for reference
Expected_CollapseOrExpandIcon.mp4
13.5 MB View Download
Expected_CollapseOrExpandIcon.jpg
341 KB View Download
Status: WontFix (was: Assigned)
This is expected. This icon is seen only on expandable notifications.
Status: Assigned (was: WontFix)
C#2>
For user which has Sync Passphrase ,Sync Error Notification is expandable notification.
In collapse Notification Text is "Chrome OS could not sync your data. please..."  (Please refer 'TextInCollpaseNotification' attachment) 
and whereas in expandable notification Text is
"Chrome OS could not sync your data. please update your Sync passphrase" 
(Please refer 'TextInExpandableNotification' attachment)
TextInCollapseNotiifcation.jpg
350 KB View Download
TextInExpandableNotification.jpg
355 KB View Download
Cc: yoshiki@chromium.org
Owner: tetsui@chromium.org
Tetsui-san, could you take a look?
Status: Fixed (was: Assigned)
I think this is already fixed in https://crrev.com/c/737503 , which landed in 64.0.3252.0
Labels: Merge-Request-63
Project Member

Comment 7 by sheriffbot@chromium.org, Nov 9 2017

Labels: -Merge-Request-63 Merge-Review-63 Hotlist-Merge-Review
This bug requires manual review: M63 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), gkihumba@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 8 by gkihumba@google.com, Nov 14 2017

Labels: -Hotlist-Merge-Review -Merge-Review-63 Merge-Approved-63
Project Member

Comment 9 by sheriffbot@chromium.org, Nov 20 2017

Cc: gkihumba@google.com
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-63 merge-merged-3239
https://crrev.com/c/776474

Sign in to add a comment