Page MenuHomePhabricator

[Regression wmf.16] Context menus for items below the screen appear broken (no vertical height)
Closed, ResolvedPublic1 Estimated Story Points

Description

Link Inspector is appearing broken on the first attempt every session.

Screen Shot 2017-03-10 at 10.52.33 AM.png (295×674 px, 54 KB)

Event Timeline

Actually, it's happening for all context menus.

Screen Shot 2017-03-10 at 11.09.22 AM.png (738×1 px, 364 KB)

Ryasmeen renamed this task from [Regression pre wmf.16] Link Inspector is appearing broken on the first attempt every session to [Regression pre wmf.16] All Inspectors and context menus are appearing broken on the first attempt every session. .Mar 10 2017, 7:12 PM
Ryasmeen renamed this task from [Regression pre wmf.16] All Inspectors and context menus are appearing broken on the first attempt every session. to [Regression pre-wmf.16] All Inspectors and context menus are appearing broken on the first attempt every session. .Mar 10 2017, 7:22 PM
Jdforrester-WMF renamed this task from [Regression pre-wmf.16] All Inspectors and context menus are appearing broken on the first attempt every session. to [Regression pre-wmf.16] Context menus for items below the screen appear broken (no vertical height).Mar 10 2017, 8:40 PM
Jdforrester-WMF triaged this task as Unbreak Now! priority.
Jdforrester-WMF set the point value for this task to 1.
Jdforrester-WMF added a subscriber: Catrope.
Ryasmeen renamed this task from [Regression pre-wmf.16] Context menus for items below the screen appear broken (no vertical height) to [Regression wmf.16] Context menus for items below the screen appear broken (no vertical height).Mar 14 2017, 11:02 PM
matmarex subscribed.

To reproduce (other than on first try):

  1. Make a context menu appear.
  2. Scroll it offscreen.
  3. Make another context menu appear.

The second context menu will always have zero height.

Change 343221 had a related patch set uploaded (by Bartosz Dziewoński):
[VisualEditor/VisualEditor] ve.ui.DesktopContext: Prevent hiding of the context popup

https://gerrit.wikimedia.org/r/343221

Change 343221 merged by jenkins-bot:
[VisualEditor/VisualEditor] ve.ui.DesktopContext: Prevent hiding of the context popup

https://gerrit.wikimedia.org/r/343221

Change 343224 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/VisualEditor] Update VE core submodule to master (ac6d7031c)

https://gerrit.wikimedia.org/r/343224

Change 343224 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor] Update VE core submodule to master (ac6d7031c)

https://gerrit.wikimedia.org/r/343224

Change 343229 had a related patch set uploaded (by Jforrester; owner: Bartosz Dziewoński):
[VisualEditor/VisualEditor] ve.ui.DesktopContext: Prevent hiding of the context popup

https://gerrit.wikimedia.org/r/343229

Change 343229 merged by jenkins-bot:
[VisualEditor/VisualEditor] ve.ui.DesktopContext: Prevent hiding of the context popup

https://gerrit.wikimedia.org/r/343229

Change 343411 had a related patch set uploaded (by Jforrester):
[mediawiki/extensions/VisualEditor] Update VE core submodule to wmf/1.29.0-wmf.16 HEAD (rGVEDf6e399eed0a2)

https://gerrit.wikimedia.org/r/343411

Change 343411 merged by jenkins-bot:
[mediawiki/extensions/VisualEditor] Update VE core submodule to wmf/1.29.0-wmf.16 HEAD (rGVEDf6e399eed0a2)

https://gerrit.wikimedia.org/r/343411

Now pushed to production; sorry to all affected.