Add tooltip text to progress bar notification |
|||||||||||||||||||||
Issue descriptionChrome Version: 65.0.3297.0/10225.0.0 dev-channel Gnawty,Peach Pit and Cyan OS: Chrome Sample URL : http://www.sample-videos.com/ What steps will reproduce the problem? (1)Sign in to user ->Now go to above URL -> download any 30 mb file (2)Now at Notifications ->Observe 'Downloading Sample video_1280*720_30m....' is seen in Expand and Collapse form and unable to identify Format(like .mp4,flv,mkv)(Please refer video & screenshot) Note : If 'Downloading Sample video_1280*720_30m....' is intended change then Tool tip should be seen on hovering mouse at Notification in Expand and Collapse form Expected: a)'Downloading Sample video_1280*720_30mb.mp4' should be seen i.e. complete text should be seen in Expand and Collapse form so that it is easy to find File Format Actual: Instead 'Downloading Sample video_1280*720_30m....' is seen and it is unable to identify File Format This is Regression Issue seen as same is working fine in 64.0.3282.24/10176.13.0 dev-channel Cyan i.e. entire Notification is seen @tetsui: Please confirm the Issue
,
Jan 31 2018
@tetsui: (1)If 'Downloading Sample video_1280*720_30m....' is intended change then Tool tip should be seen on hovering mouse at Notification in Expand and Collapse form , because to know which file format is getting downloaded need to wait until files gets downloaded (2)'Downloading Sample video_1280*720_30mb.mp4' should be seen i.e. complete text should be seen in 'Expand form' so that it is easy to find File Format (3)On Downloading any 30 mb file -> At Notifications observe 'Downloading Sample video_1280*720_30m....' is seen in 'Expand and Collapse' form and unable to identify Format(like .mp4,flv,mkv), need to wait until files gets downloaded (4)Able to reproduce the issue on latest M-66. Re-opening for tool tip behavior confirmation. Please close if this is also WAI. Attaching Screenshot for reference Thanks..!!
,
Feb 6 2018
,
Feb 7 2018
,
Feb 8 2018
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/5ac4d849fefccc9f4c310f59b796baabf5dd310a commit 5ac4d849fefccc9f4c310f59b796baabf5dd310a Author: Tetsui Ohkubo <tetsui@chromium.org> Date: Thu Feb 08 00:53:17 2018 Use custom Layout() for CompactTitleMessageView. CompactTitleMessageView is a view that shows the title an the message in one line on a notification. The view is used when the notification has a progress bar. To fit the title and the message in one line, the view elides the texts. (See the comment for the detail.) To accomplish this, previously it was manually eliding the texts in OnPaint and overwriting these labels using FillLayout. However, it has some shortcomings. * It is not the encouraged way to implement special layout. * Some accessibility features like tooltip doesn't work by default. This CL does not change the basic behavior, but uses Layout() override to achieve the special layout. TEST=manual BUG= 795721 Change-Id: I9866fe589cf93781a8bb2426fc4704da48080d72 Reviewed-on: https://chromium-review.googlesource.com/906085 Reviewed-by: Evan Stade <estade@chromium.org> Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org> Cr-Commit-Position: refs/heads/master@{#535224} [modify] https://crrev.com/5ac4d849fefccc9f4c310f59b796baabf5dd310a/ui/message_center/views/notification_view_md.cc [modify] https://crrev.com/5ac4d849fefccc9f4c310f59b796baabf5dd310a/ui/message_center/views/notification_view_md.h
,
Feb 8 2018
|
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by tetsui@chromium.org
, Dec 19 2017Status: WontFix (was: Assigned)