New issue
Advanced search Search tips

Issue 835033 link

Starred by 1 user
Status: WontFix
Owner:
kkhorimoto@chromium.org
Closed: Apr 2018
Cc:
mard...@chromium.org
ios-bugs@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: iOS
Pri: 2
Type: Bug
Q2

Blocking:
issue 835046



Sign in to add a comment

[iOS] Fulscreen Bug: Toolbars are not hidden for scrolling on AMP pages.

Project Member Reported by kkhorimoto@chromium.org, Apr 19 2018 
AMP pages are loaded in iframes, which are displayed using a separate scroll view contained within the main WKWebView scroll view.  As a result, the UIScrollViewDelegate callbacks are not received.  These delegate callbacks are used to supply information to the FullscreenModel, so we will need to experiment with a different mechanism for detecting scroll events for AMP pages.

Comment 1 by kkhorimoto@chromium.org, Apr 19 2018 

Example URL: news.google.com

List of other error conditions: go/fullscreen-error-conditions

Comment 2 by kkhorimoto@chromium.org, Apr 19 2018

Blocking: 835046

Comment 3 by pkl@chromium.org, Apr 20 2018

Cc: mard...@chromium.org
news.google.com isn't going fullscreen in Safari either, isn't it?

Comment 4 by kkhorimoto@chromium.org, Apr 20 2018 

Yes, I just verified that news.google.com doesn't go fullscreen in Safari.  Should this be marked as WontFix?  I think in order to fix this, we'd have to try to hijack touches via the WKScrollView's |panGestureRecognizer| to drive fullscreen.  However, this would be a pretty significant change in how fullscreen events are detected, so it may take a while to implement and also might introduce other regressions.

Comment 5 by cmasso@google.com, Apr 20 2018

Labels: Q2

Comment 6 by kkhorimoto@chromium.org, Apr 20 2018

Status: WontFix (was: Assigned)
Marking as WontFix since fullscreen is also broken in Safari.
►

Sign in to add a comment