Scrolling of icons in Toolbar

Bug #1558018 reported by Tim Peeters on 2016-03-16
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Ubuntu UX
High
Femma
ubuntu-ui-toolkit (Ubuntu)
Medium
Tim Peeters

Bug Description

Currently, when the icons do not all fit in the toolbar, an overflow button is shown that shows an overflow popover/panel when clicked. This needs to be replaced by icon scrolling instead.

See https://docs.google.com/document/d/1F-zD6CF8YFfA-nAtpdRniCqwUNJhSDJVFrRO76A1M-Q/edit

UX solution:
https://docs.google.com/document/edit?hgd=1&id=1F-zD6CF8YFfA-nAtpdRniCqwUNJhSDJVFrRO76A1M-Q#heading=h.w16v0yn510iy is this not what you need?

https://docs.google.com/document/edit?hgd=1&id=1F-zD6CF8YFfA-nAtpdRniCqwUNJhSDJVFrRO76A1M-Q#heading=h.duaef4xy7sqv

Related branches

Tim Peeters (tpeeters) on 2016-03-16
Changed in ubuntu-ui-toolkit (Ubuntu):
assignee: nobody → Tim Peeters (tpeeters)
importance: Undecided → High
description: updated
Femma (femma) on 2016-03-16
Changed in ubuntu-ux:
assignee: nobody → Femma (femma)
Tim Peeters (tpeeters) on 2016-03-16
description: updated
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: New → Confirmed
Tim Peeters (tpeeters) wrote :

Femma will check that the requirements are consistent and up-to-date.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → Incomplete
Tim Peeters (tpeeters) wrote :

Attached is a screenshot of how the toolbar currently looks when there are too many icons to fit. On the right-hand side it shows an overflow button, and tapping that reveals the overflow panel with the remaining actions.

Femma (femma) wrote :

Hi Tim, could you point me to exactly where in the spec did it say that the toolbar should have an overflow? I'm currently considering the scrolling vs overflow options.

Femma (femma) wrote :

I've had a look through the spec and here it shows scrolling in the toolbar: https://docs.google.com/document/edit?hgd=1&id=1F-zD6CF8YFfA-nAtpdRniCqwUNJhSDJVFrRO76A1M-Q#heading=h.w16v0yn510iy is this not what you need?

Femma (femma) on 2016-03-17
Changed in ubuntu-ux:
importance: Undecided → High
status: New → Fix Committed
description: updated
Tim Peeters (tpeeters) on 2016-03-17
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Incomplete → Confirmed
Tim Peeters (tpeeters) on 2016-05-19
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → In Progress
Tim Peeters (tpeeters) wrote :

Visuals need to be updated (discussed with Jamie)

Changed in ubuntu-ux:
status: Fix Committed → In Progress
Tim Peeters (tpeeters) on 2016-11-23
Changed in ubuntu-ui-toolkit (Ubuntu):
importance: High → Medium
Tim Peeters (tpeeters) wrote :

Updates TBD are listed in a comment of the MR.

Changed in ubuntu-ui-toolkit (Ubuntu):
status: In Progress → Confirmed
Tim Peeters (tpeeters) on 2017-01-27
Changed in ubuntu-ui-toolkit (Ubuntu):
status: Confirmed → Fix Committed
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.2166+17.04.20170130

---------------
ubuntu-ui-toolkit (1.3.2166+17.04.20170130) zesty; urgency=medium

  [ Andrea Bernabei ]
  * ListItemLayout documentation: fix typo in color usage of a code example.
    Fixes LP: #1561861

  [ Zsombor Egri ]
  * Get tst_listitem_focus_13 back alive. Fixes LP: #1624331.

  [ Arthur Mello ]
  * Add support for interacting with Content Hub Clipboard UI via DBus calls.
    Fixes LP: #1563440.

  [ Daniel d'Andrada ]
  * If GRID_UNIT_PX set, always use it. It should override scaling information
    from the QPA.

  [ Adnane Belmadiaf ]
  * Expose append method in TextArea. Fixes LP: #1658121.

  [ Tim Peeters ]
  * Toolbar visual refresh to scroll the icons instead of using an overflow
    panel. Fixes LP: #1558018.
  * Add missing dependency for the UITK gallery. Fixes lLP: #1640135

  [ Sergio Cazzolato ]
  * This change is done to remove the dependencies with upstart in the tests.
    To start apps with env vars is gonna be used the launch_test_application
    method provided by autopilot in its test base class which is a wrapper for
    the ubuntu-app-launch tools (API). This change removes all the dependencies
    with upstart, so all the projects which are using the initctl api and
    fixtures will be impacted. An update in the documentation or in the blog is
    gonna be done also to communicate this change to the developers.

  [ Christian Dywan ]
  * Add visualRoot property to MainWindow. Fixes LP: #1504551, LP: #1656857.
  * Add python3-debian to control.gles and harden package-sorting.sh.
  * Add organizationName property to MainWindow
  * Add MainWindow to Ubuntu.Components.Labs.
    Fixes LP: #1573118, LP: #1587431, LP: #1647415

  [ Albert Astals Cid ]
  * Labs is at 1.0.
  * You can't save the focus after the popup/dialog has been created, at that
    stage the dialog has focus already, you need to do it before creating it.

 -- Zoltán Balogh <email address hidden> Mon, 30 Jan 2017 11:30:03 +0000

Changed in ubuntu-ui-toolkit (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers