Issue metadata
Sign in to add a comment
|
Regression : Tool tip is seen missing for long text Default notifications |
||||||||||||||||||||||
Issue descriptionChrome Version: 62.0.3202.82/9901.66.0 stable-channel Daisy,Candy and Reks 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 Notification is in collapsed form 2)Now hover mouse pointer at Notification in collapsed form and observe for Tool tip of passphrase notification (Please refer screenshot) Expected: On hovering mouse pointer Tool tip should be seen for long text Default notification even in collapsed form Actual: Instead tool tip is seen missing This is Regression Issue seen as same is working fine in M-61 i.e. entire Notification is seen @Yoshiki: Please confirm the Issue Note : 1.Save any page with long text -> Now at Notifications -> collapse it and now hover on it and observe Tool tip (Please refer 'Expected_SavedFilesWithLongTextAndOnHoverngTooltipIsSeen' Video and Screenshot) 2.Issue is seen on latest M-64 also
,
Nov 1 2017
Able to reproduce the issue on 62.0.3202.82/9901.66.0 - Peppy steps : 1. install Chrome extensions from Chrome Web store(ex: Android Calculator, https://chrome.google.com/webstore/detail/android-calculator/jkiblcnjkgohckbbjpedihcfjanbfoif?utm_source=chrome-app-launcher-info-dialog 2. open Task manager and kill the extension 3. hover mouse pointer at Notification and check the tool tip (image1.png)
,
Nov 3 2017
I would add a tooltip to a native notification, but I'm not sure this is even possible on ARC notification technically (I suppose it's not). Also FYI, in M61 the entire message was shown but there was no tooltip, and from M62 we have expand/collapse feature instead of showing the entire message, so formally this is not a regression.
,
Nov 3 2017
,
Nov 8 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/91d33f28458664dac4ae08908d0fea9483ad9b94 commit 91d33f28458664dac4ae08908d0fea9483ad9b94 Author: Tetsui Ohkubo <tetsui@chromium.org> Date: Wed Nov 08 20:35:36 2017 Support tooltip for BoundedLabel. BoundedLabel does not directly inherits Label, and uses composition. This CL adds implementation of tooltip related methods to BoundedLabel, so that BoundedLabel would also have tooltip functionality similar to Label. TEST=manual BUG= 780445 Change-Id: I7eaf0a9421fe67810382307b101f567979e51d93 Reviewed-on: https://chromium-review.googlesource.com/753542 Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#514926} [modify] https://crrev.com/91d33f28458664dac4ae08908d0fea9483ad9b94/ui/message_center/views/bounded_label.cc [modify] https://crrev.com/91d33f28458664dac4ae08908d0fea9483ad9b94/ui/message_center/views/bounded_label.h
,
Nov 8 2017
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by yoshiki@chromium.org
, Nov 1 2017Owner: tetsui@chromium.org