Page MenuHomePhabricator

The arrow for row operations on table is missing
Closed, ResolvedPublic

Description

  1. Insert a table
  2. Select a cell

The arrow marker for the row operations is missing.

Screen Shot 2018-03-22 at 5.35.29 PM.png (586×697 px, 61 KB)

Event Timeline

We were using indicator-next which was deprecated a while ago and removed in OOUI 0.26.0. Oops.

@Volker_E suggests resizing the next and down icons instead. Bit hack but it works.

Volker_E moved this task from Backlog to Doing on the OOUI board.
This comment was removed by Esanders.

Change 421520 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Use icon instead of indicator in TableLineContext

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

Change 421520 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Use icon instead of indicator in TableLineContext

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

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

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

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

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

matmarex removed a project: OOUI.

This is also missing for mobile VE, even in production. So that's probably a separate existing issue, should I file a separate task for it?

Thanks, we can track that with this task.

Fix is working locally on mobile:

image.png (380×453 px, 18 KB)

Interesting, I see the arrow in an android device (Nexus 7) but not while using desktop browsers .