Page MenuHomePhabricator

Table row context menu appears too far below
Closed, ResolvedPublic1 Estimated Story Points

Description

Table row context menu appears too far below. The highest the table row is, the further down from the correct position the menu appears.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 20 2017, 11:59 AM

Change 354679 had a related patch set uploaded (by Bartosz Dziewoński; owner: Bartosz Dziewoński):
[VisualEditor/VisualEditor@master] ve.ui.TableLineContext: Fancier popups

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

@Jdforrester-WMF says that the above patch works in standalone VE, but makes it worse in MediaWiki. @matmarex, do you plan on addressing these issues any time soon? :-)

Yes, I've been working on some higher priority issues (this is mostly just a cosmetic problem) but I'll look into it now.

Change 354679 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] ve.ui.TableLineContext: Fancier popups

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

Change 361940 had a related patch set uploaded (by Jforrester; owner: Jforrester):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (b528a5321)

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

Change 361890 had a related patch set uploaded (by Jforrester; owner: DLynch):
[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (b528a5321)

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

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

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

Jdforrester-WMF closed this task as Resolved.Jun 29 2017, 7:28 AM
Jdforrester-WMF triaged this task as Medium priority.
Jdforrester-WMF set the point value for this task to 1.
Jdforrester-WMF moved this task from To Triage to TR1: Releases on the VisualEditor board.
Jdforrester-WMF edited projects, added VisualEditor-Tables; removed Patch-For-Review.
Restricted Application added a project: User-Ryasmeen. · View Herald TranscriptJun 29 2017, 7:28 AM

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

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