Last modified: 2014-10-16 22:59:57 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 T73616, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 71616 - OOjs UI: Toolbar menus should use sub-groups with a separator (<hr>?) between them for splitting up longer lists
OOjs UI: Toolbar menus should use sub-groups with a separator (<hr>?) between...
Status: ASSIGNED
Product: OOjs UI
Classification: Unclassified
General (Other open bugs)
unspecified
All All
: Normal enhancement
: ---
Assigned To: Bartosz Dziewoński
:
Depends on: 72159
Blocks:
  Show dependency treegraph
 
Reported: 2014-10-03 17:47 UTC by James Forrester
Modified: 2014-10-16 22:59 UTC (History)
2 users (show)

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


Attachments

Comment 1 Bartosz Dziewoński 2014-10-07 12:00:23 UTC
This looks like it'd require us to implement either subgroups of some kind, or a way to explicitly order the groups (to be able to insert separators at arbitrary positions).

If we also want sub-headings as shown in the mockup, we probably actually want the subgroup approach. Do we?
Comment 2 James Forrester 2014-10-07 16:10:38 UTC
(In reply to Bartosz Dziewoński from comment #1)
> This looks like it'd require us to implement either subgroups of some kind,
> or a way to explicitly order the groups (to be able to insert separators at
> arbitrary positions).
> 
> If we also want sub-headings as shown in the mockup, we probably actually
> want the subgroup approach. Do we?

I was thinking sub-groups, yes (hence the title ;-)). I don't think we'd ever actually use headings of sub-groups in practice, but others might.

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


Navigation
Links