preview combo should autoscroll to make sure its child buttons are on screen

Bug #1316660 reported by Michał Karnicki on 2014-05-06
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
unity8 (Ubuntu)
Medium
Albert Astals Cid

Bug Description

I have a preview, with 3 buttons. foo, bar, and baz. Because there's more than 2, they're collapsed to "More..." and "foo".

If the preview is large enough, the buttons are located at the bottom of the screen. When I press "More...", bar and baz overflow the bottom of the screen.

Expected:
Screen automatically scrolls so I can see the active, expanded component.

Actual:
I have to manually scroll the screen to see the expanded buttons.

Solution:
Currently the button combo is a hack. Expandable components from the SDK should be used instead.

Related branches

Michał Sawicz (saviq) on 2014-05-06
summary: - Scope preview allows buttons overflow bottom edge
+ Scope preview should use SDK's Expandable*s
Changed in unity8:
status: New → Triaged
importance: Undecided → High
assignee: nobody → Andrea Cimitan (cimi)
Michał Karnicki (karni) on 2014-07-01
tags: added: scope-dev
Thomas Strehl (strehl-t) on 2014-07-29
tags: added: rtm14
Michał Sawicz (saviq) on 2014-08-06
Changed in unity8:
status: Triaged → In Progress
assignee: Andrea Cimitan (cimi) → Albert Astals Cid (aacid)

Saviq, i don't think the branch you linked has anything to do with this bug, the bug is about previews with more than 2 buttons (aka PreviewActions.qml using a PreviewActionCombo.qml) nothing to do with expandables preview widgets the linked branch is about.

Michał Sawicz (saviq) on 2014-08-07
Changed in unity8:
status: In Progress → Triaged
Michał Sawicz (saviq) wrote :

The expandable preview widget landed now, it's not yet using SDK's Expandable, but that's lower priority than the feature itself.

Changed in unity8:
importance: High → Medium
tags: removed: rtm14
Michał Karnicki (karni) on 2014-08-20
summary: - Scope preview should use SDK's Expandable*s
+ preview combo should autoscroll to make sure its child buttons are on
+ screen
Michał Sawicz (saviq) on 2014-09-03
Changed in unity8 (Ubuntu):
assignee: nobody → Albert Astals Cid (aacid)
importance: Undecided → Medium
status: New → Triaged
Michał Sawicz (saviq) on 2014-09-09
no longer affects: unity8
Changed in unity8 (Ubuntu):
status: Triaged → In Progress
Albert Astals Cid (aacid) wrote :

This was already fixed a while ago but i'm now working in a test that proves it.

Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.02+15.04.20150216.1-0ubuntu1

---------------
unity8 (8.02+15.04.20150216.1-0ubuntu1) vivid; urgency=medium

  [ Andrea Cimitan ]
  * Fix temp scopes opening temp scopes, correctly close previously
    opened temp scope with its preview (LP: #1410337)
  * Set sourcesize for scopes images in manage dash

  [ CI Train Bot ]
  * Resync trunk

  [ Albert Astals Cid ]
  * Test for bug #1316660 (LP: #1316660)
  * Hardcode tranlsation for internal.location field (LP: #1393438)
  * Make sure the height it's the height it will have

  [ Michael Zanetti ]
  * Cleanup cmake warning about missing Qt5Sql module
 -- CI Train Bot <email address hidden> Mon, 16 Feb 2015 13:14:26 +0000

Changed in unity8 (Ubuntu):
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers