Last modified: 2014-06-16 17:18:07 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T46591, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 44591 - Add a label or icon to the page tools cactions dropdown menu in Vector
Add a label or icon to the page tools cactions dropdown menu in Vector
Status: RESOLVED FIXED
Product: MediaWiki
Classification: Unclassified
Interface (Other open bugs)
1.21.x
All All
: Normal enhancement (vote)
: 1.24.0 release
Assigned To: Bartosz Dziewoński
: design
Depends on:
Blocks: 44881
  Show dependency treegraph
 
Reported: 2013-02-01 19:36 UTC by Isarra
Modified: 2014-06-16 17:18 UTC (History)
11 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
Screenshot showing the change in the interface (before-after) (18.06 KB, image/png)
2014-05-31 20:04 UTC, Tomasz W. Kozlowski
Details
screenshot (125.41 KB, image/png)
2014-05-31 20:30 UTC, Quiddity
Details
Screenshot showing the change in the interface (before-after) (17.96 KB, image/png)
2014-06-04 19:02 UTC, Bartosz Dziewoński
Details

Description Isarra 2013-02-01 19:36:00 UTC
The cactions dropdown in the vector skin currently is only represented by the little arrow dropdown icon. While this means it doesn't indicate what is in this dropdown, more importantly it means it is also very easy for it to go unnoticed due to its visual similarity to the surrounding elements, thus often preventing new users from realising what tools they have at their disposal.

Adding a label saying what it is or even just a little gear or menu icon would make it stand out more, which should in turn help to alleviate the issue some.
Comment 1 Krinkle 2013-02-03 05:50:34 UTC
cc @TrevorParscal who was part of the team that designed the toolbar.
Comment 2 Bartosz Dziewoński 2014-02-08 13:24:57 UTC
Trevor's Apex skin has "Actions" next to the dropdown, we could try doing the same in Vector.
Comment 3 Gerrit Notification Bot 2014-02-08 14:53:03 UTC
Change 112236 had a related patch set uploaded by Bartosz Dziewoński:
Show a label next to Vector's dropdown actions menu

https://gerrit.wikimedia.org/r/112236
Comment 4 Gerrit Notification Bot 2014-05-31 18:51:15 UTC
Change 112236 merged by jenkins-bot:
Vector: Show a label next to the actions dropdown menu

https://gerrit.wikimedia.org/r/112236
Comment 5 Nemo 2014-05-31 18:51:31 UTC
Thank you very much!
Comment 6 Bartosz Dziewoński 2014-05-31 19:01:01 UTC
Suddenly merged. Yay!
Comment 7 Tomasz W. Kozlowski 2014-05-31 20:04:23 UTC
Created attachment 15540 [details]
Screenshot showing the change in the interface (before-after)
Comment 8 Quiddity 2014-05-31 20:30:50 UTC
Created attachment 15541 [details]
screenshot

There's a problem with the text being partially hidden, if the feature is triggered due to window-width. (See screenshot). Reproduced in firefox, opera, and chrome, whilst logged in and out.
(I'm not sure if this bug should be re-opened, or a new one started. I'll leave that to better minds.)
Comment 9 Bartosz Dziewoński 2014-05-31 21:03:11 UTC
This only happens if the "Actions" dropdown disappears completely (it has no items inside), and then something gets moved inside it causing it to reappear.

Under the default (and WMF) configuration this only affects anonymous and
non-autoconfirmed users, which is probably why I failed to notice this in my tests :( I'll submit a followup shortly.
Comment 10 Gerrit Notification Bot 2014-05-31 21:20:59 UTC
Change 136601 had a related patch set uploaded by Bartosz Dziewoński:
Vector: Fix conflicts of "Actions" dropdown with collapsibleTabs

https://gerrit.wikimedia.org/r/136601
Comment 11 Bartosz Dziewoński 2014-05-31 21:24:09 UTC
This should do it. Sorry.

Another thing I noticed is that the width of the "Actions" dropdown is barely smaller than the width of the "View history" tab, which is the last one to collapse (and could conceivably be larger in other languages). The dropdown itself should probably "fold" to just the down-arrow (hiding the text) if it can't fit on the line. But that's something for another bug and another patch. :)
Comment 12 James Forrester 2014-05-31 22:45:33 UTC
Ick. Labelling the menu "actions" makes it sound like the rest of the tabs aren't actions… maybe "more actions" (or just "more") or similar would make sense?
Comment 13 Bartosz Dziewoński 2014-05-31 22:52:14 UTC
I'm not attached to it, I felt it was good enough, but do submit improvements :)
There were some comments about the wording on the changeset too, but no one proposed a solution to make it better.

I didn't come up with the text – that label was there before, it was just hidden visually and I unhid it.
Comment 14 Bartosz Dziewoński 2014-05-31 23:02:05 UTC
On an unrelated note, if anyone feels personally offended by this change,
here's the CSS to revert it:

  #p-cactions-label { width: 24px; }
  #p-cactions-label span { display: none; }
Comment 15 Gerrit Notification Bot 2014-05-31 23:14:28 UTC
Change 136610 had a related patch set uploaded by Jforrester:
Vector: Label the more actions menu "More", not "Actions".

https://gerrit.wikimedia.org/r/136610
Comment 16 Gerrit Notification Bot 2014-06-01 05:10:29 UTC
Change 136601 merged by jenkins-bot:
Vector: Fix conflicts of "Actions" dropdown with collapsibleTabs

https://gerrit.wikimedia.org/r/136601
Comment 17 Eduard Braun 2014-06-02 09:12:34 UTC
Just read this in tech news and I'm afraid to say that this was a bad decision, sorry.

First of all as James Forester claims in comment 12 all of the buttons (especially the uncollapsed ones) are "actions". Therefore the label is even more misleading than no label at all and and therefore adds unnecessary noise to the UI.

Even if we chose a more appropriate label (e.g. the suggested "more" it's kinda redundant: People are used to the feature that tools which do not fit into the toolbar are hidden behind a small icon at the end of the toolbar. Almost every software does this! Therefore for the majority of people the triangle is crystal clear!

Finally if you want to argue that there are still some people who do not find those additional actions "hidden" behind an arrow: Remember that those actions in the drop-down are *advanced* features. Most users never need them! And even then, do we really want users to use those advanced tools, if they are not even able to *find* them to start with?


All in all regarding the "monstrous" size that our toolbar already has I'd rather leave the label out as it always was. I assume that as soon as Winter is coming all those shiny labels will be obsolete anyway...
Comment 18 Bartosz Dziewoński 2014-06-02 10:36:25 UTC
(In reply to Eduard Braun from comment #17)
> Even if we chose a more appropriate label (e.g. the suggested "more" it's
> kinda redundant: People are used to the feature that tools which do not fit
> into the toolbar are hidden behind a small icon at the end of the toolbar.
> Almost every software does this! Therefore for the majority of people the
> triangle is crystal clear!

You have a point here, but the toolbar and the icon usually look completely different, and I think that toolbars not fitting on the screen happen very rarely in desktop programs (I can't remember the last time I saw this myself). While a majority of users might understand this easily, it seems that there is still a sizable minority that doesn't (judging by the number of people who do copy-paste moves, and judging by how many times I've had to explain myself where the option is hidden).


> Finally if you want to argue that there are still some people who do not
> find those additional actions "hidden" behind an arrow: Remember that those
> actions in the drop-down are *advanced* features. Most users never need
> them! And even then, do we really want users to use those advanced tools, if
> they are not even able to *find* them to start with?

Moving pages is not a very advanced feature. And yes, we definitely want users to use it instead of copy-pasting page contents (even if they completely botch the page move, it's easier to fix than a copy-paste move).


If the text bothers you, I provided a CSS snippet to hide it in comment 14.
Comment 19 Gerrit Notification Bot 2014-06-04 19:02:30 UTC
Change 136610 merged by jenkins-bot:
Vector: Label the more actions menu "More", not "Actions"

https://gerrit.wikimedia.org/r/136610
Comment 20 Bartosz Dziewoński 2014-06-04 19:02:52 UTC
Created attachment 15566 [details]
Screenshot showing the change in the interface (before-after)

Updated screenshot for new tab text ("More" instead of "Actions").
Comment 21 James Forrester 2014-06-04 19:03:31 UTC
Deeming this now fixed. Thanks all!
Comment 22 Erwin Dokter 2014-06-12 22:46:27 UTC
This has broken every gadget and user script (including Twinkle!) that depend on the Vector tabs HTML structure. This should have been done without changing the structure. Suggest revert.
Comment 23 Matthew Flaschen 2014-06-13 04:18:57 UTC
(In reply to Erwin Dokter from comment #22)
> This has broken every gadget and user script (including Twinkle!) that
> depend on the Vector tabs HTML structure. This should have been done without
> changing the structure. Suggest revert.

It's not always possible to change the appearance of the page without noticeably changing the structure.  Thus, I don't know if it's possible to make things much easier on userscript writers while still accomplishing this particular UI change.  I'll leave it to the people who worked on the bug to say.

The original issue has been fixed, so re-marking as such.  If a clear enhancement in this regard (reducing HTML structure change in a particular way/ways) is identified, a separate bug can be opened.
Comment 24 Bartosz Dziewoński 2014-06-13 04:57:49 UTC
(In reply to Erwin Dokter from comment #22)
> This has broken every gadget and user script (including Twinkle!) that
> depend on the Vector tabs HTML structure. This should have been done without
> changing the structure. Suggest revert.

This *was* done without changing the structure. Can you explain the issue better?
Comment 25 Bartosz Dziewoński 2014-06-13 05:21:41 UTC
I figured it out myself and submitted a patch for Twinkle: https://github.com/azatoth/twinkle/pull/225. It involves removing three separate weird workarounds that now break stuff.
Comment 26 Erwin Dokter 2014-06-13 07:56:42 UTC
(In reply to Bartosz Dziewoński from comment #24)
> This *was* done without changing the structure. Can you explain the issue
> better?

The CSS had a MAJOR restructure; I can't even fix my gadget because I always end up with a double- or NO border. This was the unnecessary part I was referring to.
Comment 27 Bartosz Dziewoński 2014-06-16 16:10:04 UTC
Oh, the CSS, yes.

Sorry about that, I found it impossible to extend the old one in a way that would keep working with IE 6 and RTL languages, so I was forced to rewrite it a bit.

The trick used to ensure the dropdown is at least as wide as the tab itself https://gerrit.wikimedia.org/r/#/c/112236/13..14/skins/vector/components/tabs.less,cm is also a bit ugly, but I found no better way to do this. 

If you had possibly shared with us which gadget are you unable to fix, someone might be able to help. I've drained out my omniscience powers for the week fixing the Twinkle bug.
Comment 28 Erwin Dokter 2014-06-16 17:18:07 UTC
Anyone who wants to take a stab at it: https://test.wikipedia.org/wiki/User:Edokter/MenuTabsToggler.css

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links