New issue
Advanced search Search tips

Issue 814509 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Feature


Show other hotlists

Hotlists containing this issue:
Modern-Media-Controls


Sign in to add a comment

Modern Media Controls: Enable download when preload=none

Project Member Reported by steimel@chromium.org, Feb 21 2018

Issue description

https://gallery.googleplex.com/projects/MCHbtQVoQ2HCZZIx9p6vbtpx/files/MCHtA7U1iMGr69laPZDCC2xNn8G12BWoPZA

This also means we'll need to show the overflow menu before preload. We will only show the overflow menu before preload if download is enabled (since we'll only show it if something is enabled).
 
Cc: amyroberts@chromium.org
Status: Started (was: Assigned)
Attached are screenshots from patchset 2 of crrev.com/c/930334. Note that the dark scrim, disabled CC/pip buttons, overflow animation, etc will be dealt with in separate CLs.
crbug.814509.patchset2.normal.png
9.7 KB View Download
crbug.814509.patchset2.overflow_open.png
12.5 KB View Download
crbug.814509.patchset2.download_disabled.png
11.0 KB View Download
Labels: -Pri-3 M-67 Pri-2
Do we want to show the timeline even if the user can't interact with it?
Here are screenshots from patchset 3. It removes the thumb and has updated scrim/timeline. This still does not show disabled fullscreen/mute buttons or other disabled overflow options, but those will be taken care of in later CLs.
crbug.814509.patchset3.normal.png
9.6 KB View Download
crbug.814509.patchset2.overflow_open.png
12.4 KB View Download
crbug.814509.patchset2.download_disabled.png
9.1 KB View Download
Project Member

Comment 4 by bugdroid1@chromium.org, Mar 19 2018

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

commit 5787b01ca784633e937c7f0cdef5dc93d236035a
Author: Tommy Steimel <steimel@chromium.org>
Date: Mon Mar 19 19:12:27 2018

Show the timeline and overflow menu before preload

This CL changes the CSS to show the timeline and overflow button before
preload on the modern media controls. This exposes the download button
so that the user can download a video without preloading.

Bug:  814509 
Change-Id: I7993204dc6d0bc38ef77e18a14ae1cfa07ab3ca8
Reviewed-on: https://chromium-review.googlesource.com/930334
Reviewed-by: Becca Hughes <beccahughes@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Commit-Queue: Tommy Steimel <steimel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#544103}
[modify] https://crrev.com/5787b01ca784633e937c7f0cdef5dc93d236035a/third_party/WebKit/Source/modules/media_controls/resources/modernMediaControls.css

Status: Fixed (was: Started)

Sign in to add a comment