Page MenuHomePhabricator

iOS Safari: Can't save edit in WikiText editor after entering edit summary
Closed, ResolvedPublic


Steps to reproduce:

  1. On an iPhone, start editing an article with mobile Safari in the WikiText editor (can be either logged in or logged out)
  2. Click the "Next" button
  3. Click in the text area to write an edit summary (which should activate the virtual keyboard).
  4. Hit "Done" to dismiss the virtual keyboard.

Expected result: You can now click "Save" and finish the edit.
Actual result: The "Save" button isn't visible and you can't scroll further up than the "How did you improve the page?" text. If you pinch to zoom out, however, you can get to the button and save.

Here's a screenshot. You can't scroll up further than this. Notice also the grey line near the bottom of the screen. Not sure if that's related.

Browser details: Mobile Safari, iOS 12.1, iPhone SE

May be related to T218062, T211490, and/or T218069, but Rummana suggested I file a separate bug just in case.

Event Timeline

kaldari created this task.Mar 14 2019, 7:47 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 14 2019, 7:47 PM
kaldari updated the task description. (Show Details)Mar 14 2019, 7:52 PM

Here's a graph Ed generated of number of saveSuccess events in iOS. Looks like there was a noticeable drop around Feb 20th, which might be related to this bug. David Lynch hypothesizes that the bug mainly affects iPhones with smaller screens.

Change 496682 had a related patch set uploaded (by Jdlrobson; owner: Jdlrobson):
[mediawiki/extensions/MobileFrontend@master] iOS: Do not jump to top of text editor during typing

Change 496682 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Do not jump to top of text editor during typing

For future reference, in case we go back to the previous approach (@Jdlrobson's commit here basically removes all iOS scrolling workarounds), I think the necessary fix would be to change the EditorOverlay isActiveWithKeyboard() function:

	isActiveWithKeyboard: function () {
		return this.$( '.wikitext-editor' ).is( ':focus' );

To instead read:

		return this.$( '.wikitext-editor' ).is( ':focus' ) || this.$( '.summary' ).is( ':focus' );

(I didn't test this though)

Stashbot added a subscriber: Stashbot.

Mentioned in SAL (#wikimedia-operations) [2019-03-15T17:59:54Z] <thcipriani@deploy1001> Synchronized php-1.33.0-wmf.21/extensions/MobileFrontend: SWAT: [[gerrit:496827|iOS: Fix mobile editor]] T218069 T218062 T218352 T211490 T218062 T211491 T172877 (duration: 00m 54s)

This should be fixed in production. Can somebody confirm?

Jdlrobson moved this task from Incoming to QA on the VisualEditor (Current work) board.
ppelberg closed this task as Resolved.Mar 15 2019, 8:19 PM
ppelberg claimed this task.
Ryasmeen reassigned this task from ppelberg to Jdlrobson.Mar 15 2019, 9:20 PM