Page MenuHomePhabricator

Design review for UrlShortener
Closed, ResolvedPublic

Description

Interfaces to review

Tool sidebar

ABC
Screen Shot 2018-04-06 at 9.32.17 AM.png (372×302 px, 37 KB)
Screen Shot 2018-04-06 at 9.32.19 AM.png (364×300 px, 35 KB)
Screen Shot 2018-04-06 at 9.32.19 AM 1.png (381×303 px, 39 KB)

Special page

ABC
Screen Shot 2018-04-06 at 9.37.20 AM.png (390×1 px, 48 KB)
Screen Shot 2018-04-06 at 9.39.13 AM.png (382×1 px, 57 KB)
Screen Shot 2018-04-06 at 9.39.38 AM.png (368×1 px, 68 KB)
DE
Screen Shot 2018-04-06 at 9.39.49 AM.png (536×1 px, 89 KB)
Screen Shot 2018-04-06 at 9.40.36 AM.png (232×1 px, 25 KB)

Event Timeline

Prtksxna created this task.

Nope, I think that's it :)

I'm going to add a moderation interface as part of T133109: Add basic abuse prevention to UrlShortener but that's going to be admin-only and it'll just use the standard OOUIHTMLForm so it should be trivial to review as part of the patch itself.

Here is a quick review of these screens, we can discuss which of these, if any, need to become blockers, or tasks to resolve later.

  • Tool sidebar
    • A & B:
    • C: Maybe we want to show the URL in an OOUI Popup, like Wikidata Query, and change the link to Shorten another URL which takes you to the special page.
  • Special Page
    • A & B:
    • C: Error message too long, obscuring the input. Maybe use notice instead of inline label.
    • D: Too many notices, let's have one either for the form or the input. I prefer input.
    • E: Does this need a screen of it's own? Would it be better to show it on the same page like in -https://upload.wikimedia.org/wikipedia/commons/a/a7/UrlShortener.png

I'm going to add a moderation interface as part of T133109: Add basic abuse prevention to UrlShortener but that's going to be admin-only and it'll just use the standard OOUIHTMLForm so it should be trivial to review as part of the patch itself.

Yep, sounds good.

(my2c, I really like the sidebar option, but I am aware that some don't ever want that touched. Just noting the option should be closer anyway to Permanent link, imho. If only there was a way to put both on the same line. )

Change 496877 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/UrlShortener@master] Move showing errors to below the input

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

Change 496877 had a related patch set uploaded (by Ladsgroup; owner: Ladsgroup):
[mediawiki/extensions/UrlShortener@master] Move showing errors to below the input

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

Change 496877 merged by jenkins-bot:
[mediawiki/extensions/UrlShortener@master] Move showing errors to below the input

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

Ladsgroup subscribed.

I have implemented most of the ideas. We are going to close this.