Page MenuHomePhabricator

Convert Special:Import to OOUI
Closed, ResolvedPublic

Description

Example on beta labs.

  • Transform to OOUI (including few tricky bits here):
    • HTMLForm currently doesn't support the fancy mix of radio buttons, combo boxes and text boxes, as seen in the "new" section of M28. While I could change this to use hide-if, that would force an ugly vertical layout; makes it difficult to write good radio button labels; and is also confusing to the user (they can't see the relevant field until they select the option). I'd prefer to keep the current layout, but possibly using a conditional disable instead of conditional hiding.
    • Also apparently HTMLForm doesn't support two forms on one page (T102114), but HTMLForm is sophisticated enough that I suspect there might be a way around this.
    • How to deal with the file upload on PHP vs JS side? T170263
Before After
image.png (800×2 px, 145 KB)
image.png (1×2 px, 162 KB)

Details

Event Timeline

TTO claimed this task.
TTO raised the priority of this task from to Low.
TTO updated the task description. (Show Details)
TTO subscribed.
Legoktm subscribed.

I started work on this but can't finish until these issues are resolved.

Can you file blockers for the other issues? :)

Volker_E renamed this task from Convert Special:Import to use HTMLForm to Convert Special:Import to OOUI.Jan 19 2016, 12:00 AM
Volker_E set Security to None.
Volker_E moved this task from Unsorted to Conversion to OOUI on the UI-Standardization board.
TTO removed TTO as the assignee of this task.Feb 12 2016, 11:20 AM
Volker_E updated the task description. (Show Details)
Volker_E updated the task description. (Show Details)
Volker_E added a subscriber: matmarex.
This comment was removed by DannyS712.
DannyS712 raised the priority of this task from Low to Medium.Jan 30 2020, 6:30 PM
DannyS712 moved this task from Next to In progress on the User-DannyS712 board.

Change 645439 had a related patch set uploaded (by Esanders; owner: Esanders):
[mediawiki/core@master] Convert Special:Import to OOUI

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

DannyS712 changed the task status from Stalled to Open.Jan 22 2021, 1:52 AM
DannyS712 removed DannyS712 as the assignee of this task.
DannyS712 added subscribers: Esanders, DannyS712.

Since there is a current patch from @Esanders, and I may not have time to work on this, unlicking

Test wiki created on Patch Demo by Kizule using patch(es) linked to this task:

https://patchdemo.wmflabs.org/wikis/831588cb7f5fe061bf7df7e6b8097d85/w/

Looks very nice, thanks! :)

image.png (1×1 px, 130 KB)

Change 645439 merged by jenkins-bot:
[mediawiki/core@master] Convert Special:Import to OOUI

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

Test wiki created on Patch Demo by Kizule using patch(es) linked to this task:

https://patchdemo.wmflabs.org/wikis/831588cb7f5fe061bf7df7e6b8097d85/w/

Looks very nice, thanks! :)

image.png (1×1 px, 130 KB)

On phone too:

Screenshot_20210204_025012_com.microsoft.emmx.jpg (2×720 px, 267 KB)

If anyone wants to test this out on the beta cluster I can grant you importer rights

On enwiki on the beta cluster, the "Import as subpages" option is selected by default for both XML and transwiki import. It should default to "Import to default locations" in the first instance, then when the form is reloaded, the option the user previously selected should be chosen.

In T108792#6802631, @TTO wrote:

On enwiki on the beta cluster, the "Import as subpages" option is selected by default for both XML and transwiki import. It should default to "Import to default locations" in the first instance, then when the form is reloaded, the option the user previously selected should be chosen.

I filed this as a Regression subtask: T274025

Please see possibly related new bug; T275031

Volker_E assigned this task to Esanders.
Volker_E updated the task description. (Show Details)