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

Issue 848216 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner:
Closed: Nov 20
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Non-Regression : Arrow pointer is seen instead of Hand Pointer on hovering mouse pointer on video seekbar

Project Member Reported by mmanchala@chromium.org, May 31 2018

Issue description

Chrome Version:  67.0.3396.67/10575.51.0 beta-channel Daisy,Candy & Reks
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 file(Ex: Download any video)
(2)Now go to Files App  -> Select downloaded video -> Right click to open context menu and select 'Open with Video Player' option
(3)Now hover mouse pointer on video seekbar and observe arrow pointer is seen instead of Hand pointer
(Please refer Video and Screenshot)

Note : Issue is also seen in General Information Page

Expected: Hand pointer should be seen on hovering mouse pointer on video seekbar
Actual: Instead Arrow pointer is seen 

This is Non-Regression Issue seen form M-62

@fukino : Please confirm the Issue
 
Actual_ArrowPointer.mp4
17.1 MB Download
Actual_ArrowPointer.jpg
823 KB View Download
Attaching Expected video and screenshot for reference
Expected_HandPointer.webm
11.4 MB View Download
Expected_HandPointer.jpg
523 KB View Download
Cc: fukino@chromium.org
Labels: -Pri-2 -M-67 CrOSFilesCategory-UI Pri-3
Owner: ----
Status: Available (was: Assigned)
Labels: Files-Fixit-2018
Owner: lucmult@chromium.org
Status: Started (was: Available)
Cc: mlamouri@chromium.org
Project Member

Comment 7 by bugdroid1@chromium.org, Nov 20

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

commit 4c48422d79696901578dd987aee50ee8dbe87e2e
Author: Luciano Pacheco <lucmult@chromium.org>
Date: Tue Nov 20 10:03:48 2018

Quick view: allow tab to audio and video controls

Change quick view for audios and videos to allow to use tab to focus
the audio and video HTML5 tag's controls, e.g.: play, slider and
volume.

Add a blue outline for focused elements inside audio and video HTML5
tags, as part of the accessibility to make visually clear what control
is focused.

Change audio to not show the download button since it's already in the
file system and downloading doesn't make sense in that context.

Change video to not show picture-in-picture since it doesn't work for
quick-view.

Bug: 627701,  801522 ,  848216 
Change-Id: Ie46159bc90d2948bbac98d69acaf8b53e62d5e58
Reviewed-on: https://chromium-review.googlesource.com/c/1341777
Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
Reviewed-by: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#609653}
[modify] https://crrev.com/4c48422d79696901578dd987aee50ee8dbe87e2e/ui/file_manager/file_manager/foreground/elements/files_quick_view.html
[modify] https://crrev.com/4c48422d79696901578dd987aee50ee8dbe87e2e/ui/file_manager/file_manager/foreground/elements/files_safe_audio_webview_content.css
[modify] https://crrev.com/4c48422d79696901578dd987aee50ee8dbe87e2e/ui/file_manager/file_manager/foreground/elements/files_safe_audio_webview_content.html
[modify] https://crrev.com/4c48422d79696901578dd987aee50ee8dbe87e2e/ui/file_manager/file_manager/foreground/elements/files_safe_media.js
[modify] https://crrev.com/4c48422d79696901578dd987aee50ee8dbe87e2e/ui/file_manager/file_manager/foreground/elements/files_safe_video_webview_content.css
[modify] https://crrev.com/4c48422d79696901578dd987aee50ee8dbe87e2e/ui/file_manager/file_manager/foreground/elements/files_safe_video_webview_content.html
[modify] https://crrev.com/4c48422d79696901578dd987aee50ee8dbe87e2e/ui/file_manager/file_manager/main.html

Status: Fixed (was: Started)

Sign in to add a comment