Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Page MenuHomePhabricator

Convert Special:NewPages to OOUI
Closed, ResolvedPublic

Assigned To
Authored By
Florian
Nov 4 2015, 5:23 PM
Referenced Files
F24361178: image.png
Jul 31 2018, 1:01 PM
F18372987: image.png
May 20 2018, 12:36 PM
F5269541: pasted_file
Jan 10 2017, 11:45 PM
F4492693: pasted_file
Sep 20 2016, 6:46 PM
F4492684: pasted_file
Sep 20 2016, 6:45 PM
F3827546: pasted_file
Apr 4 2016, 12:13 PM
Tokens
"Haypence" token, awarded by RandomDSdevel."Party Time" token, awarded by Liuxinyu970226.

Description

Already uses HTMLForm.

BeforeAfter
image.png (728×2 px, 148 KB)
image.png (970×2 px, 121 KB)

Event Timeline

Florian raised the priority of this task from to Medium.
Florian updated the task description. (Show Details)
Florian added a project: UI-Standardization.

Change 276946 had a related patch set uploaded (by Pankajeshkr):
Changed SpecialNewpages to use OOui

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

Change 276947 had a related patch set uploaded (by Pankajeshkr):
Changed Special_Newpages to use OOui

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

Change 276947 abandoned by Pankajeshkr:
Changed Special_Newpages to use OOui

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

pasted_file (539×895 px, 27 KB)

  • The hardcoded wrapper <fieldset> should probably be removed too.
  • There's no OOUI version of HTMLTagFilter.

pasted_file (539×895 px, 27 KB)

  • The hardcoded wrapper <fieldset> should probably be removed too.
  • There's no OOUI version of HTMLTagFilter.

matmarex, so this would get only fixed if we have an OOUI version of tag-filter ? Is there a task for that one, or is already done ?

Yeah. I don't know of any tasks. It's not done, but it seems fairly easy
– all the pieces are there and they just need to be connected.
ChangeTags::buildTagFilterSelector() already has an option to return
an OOjs UI widget, so HTMLTagFilter just needs to have a
getInputOOUI() method added that would call that and return the widget.

Change 311758 had a related patch set uploaded (by Florianschmidtwelzow):
HTMLForm: Implement OOUI version of HTMLTagFilter

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

Ok, the recent version I uploaded changed the changetag filter a bit, so that it suggests the filter, which are currently available, insteaad of requireing the user to type the tag in. However, because of recentchanges in the form, there's one more form element, which does not support OOUI:

pasted_file (561×910 px, 38 KB)

Change 311758 merged by jenkins-bot:
[mediawiki/core@master] HTMLForm: Implement OOUI version of HTMLTagFilter

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

Florian claimed this task.

The patch seems for converting to OOUI is not merged yet. Just saying :P

Florian removed Florian as the assignee of this task.

Ah damn :D Sorry, just saw my change merged, and I thought I uploaded one to change NewPages, too (as I saw my example image) :P Thanks for pointing to this @Ladsgroup :D

Change 276946 had a related patch set uploaded (by VolkerE; owner: Pankajeshkr):
[mediawiki/core@master] Changed Special:NewPages to use OOUI

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

Change 276946 merged by jenkins-bot:
[mediawiki/core@master] Special:NewPages: Change to use OOUI

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

Volker_E closed this task as Resolved.EditedJul 31 2018, 12:49 PM
Volker_E assigned this task to pankajeshkr.
Volker_E removed a project: Patch-For-Review.
Volker_E removed a subscriber: gerritbot.
Volker_E subscribed.

Thanks @pankajeshkr for the bunch of work here!

Change 276963 had a related patch set uploaded (by Winston Sung; author: Vishal1796):

[mediawiki/core@master] Modified SpecialNewPage

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

Change 276963 abandoned by Winston Sung:

[mediawiki/core@master] Modified SpecialNewPage

Reason:

Replaced-by: I802d215374c0b5247247dd4ae2facce251e8d369

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