Bug 114691 - MediaDocument should set a max-width on its <video> element.
Summary: MediaDocument should set a max-width on its <video> element.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Media (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Jer Noble
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-04-16 10:17 PDT by Jer Noble
Modified: 2015-01-26 08:52 PST (History)
7 users (show)

See Also:


Attachments
Patch (1.49 KB, patch)
2013-04-16 10:27 PDT, Jer Noble
eric.carlson: review+
Details | Formatted Diff | Diff
Patch for landing (1.62 KB, patch)
2014-03-18 16:47 PDT, Jer Noble
no flags Details | Formatted Diff | Diff
Patch for landing (1.58 KB, patch)
2014-03-18 16:57 PDT, Jer Noble
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jer Noble 2013-04-16 10:17:40 PDT
When a video element in a MediaDocument is too large for the browser window which contains it, the user must scroll left and right to access all the media controls.  Instead, the MediaDocument should set a CSS rule on its <video> element: "max-width: 100%".  This would scale the video side down to fit within the browser window.
Comment 1 Jer Noble 2013-04-16 10:27:34 PDT
Created attachment 198342 [details]
Patch
Comment 2 Jer Noble 2014-03-18 16:47:00 PDT
Created attachment 227130 [details]
Patch for landing
Comment 3 Jer Noble 2014-03-18 16:57:17 PDT
Created attachment 227133 [details]
Patch for landing
Comment 4 WebKit Commit Bot 2014-03-18 17:38:28 PDT
Comment on attachment 227133 [details]
Patch for landing

Clearing flags on attachment: 227133

Committed r165859: <http://trac.webkit.org/changeset/165859>