Page MenuHomePhabricator

Add a link in VE: Include "below" and "inline" position values for ve.ui.ToolbarDialog upstream
Closed, ResolvedPublic

Description

RecommendedLinkToolBarDialog class inherits from ve.ui.ToolbarDialog and uses custom position values: below for mobile and inline for desktop. These supported values should be updated and documented in VE as well.

Event Timeline

mewoph renamed this task from Add a link in VE: Include new position values upstream to Add a link in VE: Include "below" and "inline" position values for ve.ui.ToolbarDialog upstream.May 3 2021, 11:40 PM
mewoph moved this task from Backlog to May 3 - May 7 on the Add-Link board.
kostajh triaged this task as Medium priority.May 4 2021, 8:45 AM
kostajh lowered the priority of this task from Medium to Low.May 5 2021, 10:45 AM

Change 691284 had a related patch set uploaded (by MewOphaswongse; author: MewOphaswongse):

[VisualEditor/VisualEditor@master] Add positions 'inline' and 'below' to ve.ui.ToolbarDialog

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

Change 691284 merged by jenkins-bot:

[VisualEditor/VisualEditor@master] Add positions 'inline' and 'below' to ve.ui.ToolbarDialog

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

Change 694632 had a related patch set uploaded (by Bartosz Dziewoński; author: Bartosz Dziewoński):

[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (bc819fdc6)

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

Change 694632 merged by jenkins-bot:

[mediawiki/extensions/VisualEditor@master] Update VE core submodule to master (bc819fdc6)

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

kostajh moved this task from May 24 – May 28 to Done / QA on the Add-Link board.
kostajh subscribed.

Nothing to QA here, resolving.