Page MenuHomePhabricator

Add missing icons for table row/col move
Closed, ResolvedPublic

Description

These were apparently not re-designed in the overhaul.

image.png (200×154 px, 8 KB)

Event Timeline

Change 425415 had a related patch set uploaded (by VolkerE; owner: VolkerE):
[oojs/ui@master] icons: Add 'tableMoveColumn*' & 'tableMoveRow*' icons

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

Change 425415 merged by jenkins-bot:
[oojs/ui@master] icons: Add 'tableMoveColumn*' & 'tableMoveRow*' icons

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

Change 425596 had a related patch set uploaded (by Esanders; owner: Esanders):
[VisualEditor/VisualEditor@master] Remove table move icons, upstreamed to OOUI

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

Change 425596 merged by jenkins-bot:
[VisualEditor/VisualEditor@master] Remove table move icons, upstreamed to OOUI

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

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

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

Jdforrester-WMF moved this task from Next-up to OOUI-0.26.3 on the OOUI board.
Jdforrester-WMF edited projects, added OOUI (OOUI-0.26.3); removed OOUI.
Jdforrester-WMF edited projects, added VisualEditor-Tables; removed Patch-For-Review.

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

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

Esanders reopened this task as Open.EditedApr 13 2018, 5:18 PM

The icons provided in https://gerrit.wikimedia.org/r/#/c/425415/ use white shapes to mask out parts canvases. OOUI icons are only supposed to contain black and transparent, otherwise they won't work against light colour backgrounds, and they won't get inverted properly.

Change 426125 had a related patch set uploaded (by Esanders; owner: Esanders):
[oojs/ui@master] Remove white canvases from table move icons

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

Change 426125 merged by jenkins-bot:
[oojs/ui@master] Remove white canvases from table move icons

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