WebKit Bugzilla
New
Browse
Search+
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
NEW
177899
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
Details
wip patch
(3.37 KB, patch)
2018-01-08 10:17 PST
,
Adele Peterson
no flags
Details
Formatted Diff
Diff
View All
Add attachment
proposed patch, testcase, etc.
mitz
Comment 1
2017-10-04 14:33:26 PDT
<
rdar://problem/34820953
>
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.
Top of Page
Format For Printing
XML
Clone This Bug