Page MenuHomePhabricator

Convert Special:BookSources to OOUI
Closed, ResolvedPublic

Description

Looks pretty easy.

BeforeAfter
pasted_file (946×1 px, 70 KB)
pasted_file (946×1 px, 71 KB)

Event Timeline

Florian raised the priority of this task from to Medium.
Florian updated the task description. (Show Details)

https://gerrit.wikimedia.org/r/236270 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

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

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

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

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

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

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

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

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

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

Change 277050 abandoned by Jforrester:
Changed OOUI

Reason:
Parent abandoned by same author.

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

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:

BeforeAfter
pasted_file (946×1 px, 70 KB)
pasted_file (946×1 px, 71 KB)

I think it looks okay.

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

Screen Shot 2016-06-06 at 22.13.51.png (919×1 px, 197 KB)

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 https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=724187004#Special:BookSources
Excerpts:

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.