Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#33137 closed defect (bug) (fixed)

Fullscreen is broken in iOS Safari

Reported by: iseulde Owned by: azaozz
Priority: normal Milestone: 4.4
Component: TinyMCE Version: 4.3
Severity: normal Keywords:
Cc: Focuses:

Description

You can't scroll.

Attachments (3)

33137.patch​ (783 bytes ) - added by iseulde 11 years ago.
33137.1.patch​ (783 bytes ) - added by azaozz 11 years ago.
editor fullscreen nexus 5.png​ (68.6 KB ) - added by ryan 11 years ago.
Fullscreen on a Nexus 5 with some filler text to induce scrolling

Download all attachments as: .zip

Change History (13)

@iseulde
11 years ago

@azaozz
11 years ago

#1 @azaozz
11 years ago

In 33137.1.patch: disable TinyMCE fullscreen on iOS. Works well on Android and all other browsers.

This ticket was mentioned in ​Slack in #core-flow by azaozz. ​View the logs.


11 years ago

#3 @ryan
11 years ago

I see scroll bleed on my iPhone 6+. We've battled scroll bleed on iOS so often that there's a tag and glossary entry for it.

​https://make.wordpress.org/flow/tag/scroll-bleed/

​https://make.wordpress.org/flow/handbook/glossary/#scroll-bleed

Android does not usually exhibit scroll bleed. I tested fullscreen on a Nexus 5, and editor scrolling was fine.

@ryan
11 years ago

Fullscreen on a Nexus 5 with some filler text to induce scrolling

#4 @iseulde
11 years ago

IMHO, fullscreen should always be removed on phones.

This ticket was mentioned in ​Slack in #core-editor by azaozz. ​View the logs.


11 years ago

This ticket was mentioned in ​Slack in #core by azaozz. ​View the logs.


11 years ago

#7 @miqrogroove
11 years ago

  • Version → trunk

I was unable to repro bug on 4.2.4. Appears to be a regression in 4.3 RC2.

#8 @samuelsidler
11 years ago

I think fullscreen should be removed on phones too. The clutter is already hidden by nature of the small screen. :)

#9 @azaozz
11 years ago

  • Owner set to azaozz
  • Resolution → fixed
  • Status new → closed

In 33835:

TinyMCE: disable the native fullscreen plugin on mobile devices by default.
Props iseulde. Fixes #33137;

#10 @azaozz
11 years ago

  • Keywords needs-patch removed
  • Milestone Future Release → 4.4
Note: See TracTickets for help on using tickets.