Page MenuHomePhabricator

Edit interface loaded with double toolbar on mobile web
Closed, ResolvedPublic


Steps to reproduce:

  1. On your mobile device, go to
  2. Open the visual editor


  • The edit interface loads with only one toolbar


  • Two identical toolbars are loaded, and remained throughout the edit session

Screenshot_20180822-120012.png (2×1 px, 811 KB)

Screenshot_20180822-120021.png (2×1 px, 901 KB)

Event Timeline

Deskana triaged this task as Medium priority.Aug 22 2018, 11:20 AM
Deskana moved this task from To Triage to Up next on the VisualEditor board.

This seems to happen on any sufficiently large article, such as

Screenshot_20180822-123024.png (2×1 px, 494 KB)

@Deskana: I'm not seeing this on my Android device (even with 4x CPU throttling). I do see some toolbar weirdness though, as the stylesheets are loading:

2018-08-22 19.45.38.png (1×1 px, 379 KB)

I'll look into the toolbar issue shown above though - it looks like we're showing the editing surface too early on mobile.

Change 454821 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/VisualEditor@master] Don't wait for surface to blur before hiding pageToolbar

I saw the bug described locally, so I'm confident the issue I saw above is a separate issue, so I'll separate the tasks.

When I inspected the DOM, it looked like the whole editor had be initialised twice, both the toolbar and the content area. I suspect this bug is coming from MobileFrontend (perhaps the way it uses historyState to navigate), so I'll look into preventing the editor from loading twice. I suspect this may be the same cause as T202517, as I've only seen both bugs at the same time.

Change 455826 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/extensions/MobileFrontend@master] Guard against VE overlay being shown twice

Change 455826 merged by jenkins-bot:
[mediawiki/extensions/MobileFrontend@master] Guard against VE overlay being shown twice

I was able to reproduce the original issue @Deskana reported, will check both these issues once the fix lands on production.

This is going to be moved to "Done" column, once it's verified on production next week.

Not seeing two toolbars anymore. Verified.