Page MenuHomePhabricator

Convert Special:BookSources to OOUI
Closed, ResolvedPublic


Looks pretty easy.


Event Timeline

Florian raised the priority of this task from to Medium.
Florian updated the task description. (Show Details) converts SpecialBookSources from SpecialPage to FormSpecialPage.
Then, it is only a matter of changing getDisplayFormat().

Change 277049 had a related patch set uploaded (by Sukeesh):
Changed to OOUI

Change 277050 had a related patch set uploaded (by Sukeesh):
Changed OOUI

Change 277049 abandoned by Sukeesh:
Changed SpecialBookSources.php to OOUI

Change 286594 had a related patch set uploaded (by Sethakill):
Convert Special:BookSources to use OOUI

Change 286594 merged by jenkins-bot:
Convert Special:BookSources to use OOUI

Change 277050 abandoned by Jforrester:
Changed OOUI

Parent abandoned by same author.

Why is there an indicator ?
And shouldn't we make it a rule that if a button submits only a single textfield, the controls should be on the same line in wide ui's ? Seems a bit of a waste of the vertical space.

Screenshots for reference:


I think it looks okay.

On my 13" screen it's almost a third of the vertical space:

It seems considerably less efficient.
[edit] Don't mind the width, that's my own personal styling (and so is the yellow for focus)

@TheDJ I don't agree that horizontal layout of button aside of input gives any advantage here, on contrary I think, that from eye movement the vertical layout makes this specific form even easier to use.
I've already started at T136790: Improve form layouts in OOUI MW core forms for better user experience to collect some general points to work on OOjs UI form layouts. Please help identify and add overall issues there.

Further comments about this OOUI change, at

Why is the box at Special:BookSources spread over four lines instead of two, why does it have fuzzy edges? Both of these waste space, and mean that extra scrolling is now necessary where it wasn't before. It looks very much like it's Vector skin, although I use MonoBook. And what's the asterisk for?

I'm not a fan of the large space use in mw:OOUI. [...] I use Special:ExpandTemplates a lot and also find OOUI annoying there.

I'll personally add, for Special:Booksources in particular: The input field should only be 17 characters long.

  • ISBNs can only be 17 characters long (13+4 separators). We could forcibly restrict it to that many characters, but we probably want to give users room to edit (by adding and then subtracting characters) or to paste longer clipboard text and then reduce it down.