NEW177899
Elements are laid out incorrectly in full screen mode when _minimumLayoutWidth is nonzero
https://bugs.webkit.org/show_bug.cgi?id=177899
Summary Elements are laid out incorrectly in full screen mode when _minimumLayoutWidt...
mitz
Reported 2017-10-04 14:32:06 PDT
To reproduce: 1. Change MiniBrowser to set its WKWebView’s _minimumLayoutWidth to some nonzero value, such as 400 2. Load a webpage with a video element with controls 3. Click the full screen control Results: The video enters full-screen mode, but doesn’t lay out to fit in the full-screen window.
Attachments
fullscreen test (5.92 KB, text/html)
2018-01-05 11:28 PST, Adele Peterson
no flags
wip patch (3.37 KB, patch)
2018-01-08 10:17 PST, Adele Peterson
no flags
mitz
Comment 1 2017-10-04 14:33:26 PDT
Adele Peterson
Comment 2 2018-01-03 10:38:47 PST
Also occurs for other fullscreen elements, like the ones in this test page: https://davidwalsh.name/demo/fullscreen.php
Adele Peterson
Comment 3 2018-01-05 11:28:30 PST
Created attachment 330562 [details] fullscreen test
Adele Peterson
Comment 4 2018-01-08 10:17:20 PST
Created attachment 330713 [details] wip patch Adding a work in progress patch. Still trying to decide how to write a test - whether to write an API test or add something to DRT (or the WK2 equivalent).
Frédéric Wang (:fredw)
Comment 5 2018-05-23 06:59:42 PDT
Comment on attachment 330713 [details] wip patch View in context: https://bugs.webkit.org/attachment.cgi?id=330713&action=review > Adding a work in progress patch. Still trying to decide how to write a test - whether to write an API test or add something to DRT (or the WK2 equivalent). Not sure if that can really help here, but there is this bug to add fullscreen support to WebDriver: bug 180398 > Source/WebKit/UIProcess/Cocoa/WebViewImpl.h:638 > + nit: remove extra whitespace.
Note You need to log in before you can comment on or make changes to this bug.