Media Controls: Volume slider has wrong padding for larger videos |
|||
Issue descriptionLarger videos have a larger button panel, but the volume slider styling doesn't account for it. This needs to be updated
,
Oct 1
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/85757796b22f94e0a25d1998f6bd828e0843e997 commit 85757796b22f94e0a25d1998f6bd828e0843e997 Author: Tommy Steimel <steimel@chromium.org> Date: Mon Oct 01 22:17:33 2018 [Media Controls] Fix volume slider padding for larger videos This CL updates the styling for the volume slider for larger videos. Bug: 890092 Change-Id: I1b410b7318f10247457c46cadbbb081801e19acf Reviewed-on: https://chromium-review.googlesource.com/1250206 Reviewed-by: Becca Hughes <beccahughes@chromium.org> Commit-Queue: Tommy Steimel <steimel@chromium.org> Cr-Commit-Position: refs/heads/master@{#595579} [modify] https://crrev.com/85757796b22f94e0a25d1998f6bd828e0843e997/third_party/blink/renderer/modules/media_controls/resources/modernMediaControls.css
,
Oct 1
,
Oct 3
Update : --------- Verified above issue in latest Canary build #71.0.3569.0 on Windows(7, 8, 8.1, 10), Linux(14.04 LTS) and Mac(10.12.6, 10.13.1, 10.14.1, 10.13.6) OS and the issue is fixed. Volume slider is now aligned properly with 'Volume' icon and it is working as intended, hence adding TE-Verified labels. Kindly review an attached screen-cast for reference. Thank you..! |
|||
►
Sign in to add a comment |
|||
Comment 1 by steimel@chromium.org
, Sep 28