Merge lp://qastaging/~tpeeters/ubuntu-ui-toolkit/09-desktop-toolbar into lp://qastaging/ubuntu-ui-toolkit

Proposed by Tim Peeters
Status: Merged
Approved by: Zsombor Egri
Approved revision: 957
Merged at revision: 1002
Proposed branch: lp://qastaging/~tpeeters/ubuntu-ui-toolkit/09-desktop-toolbar
Merge into: lp://qastaging/ubuntu-ui-toolkit
Prerequisite: lp://qastaging/~tpeeters/ubuntu-ui-toolkit/optIn-newHeader
Diff against target: 99 lines (+33/-1)
3 files modified
components.api (+1/-0)
modules/Ubuntu/Components/Panel.qml (+29/-1)
modules/Ubuntu/Components/Toolbar.qml (+3/-0)
To merge this branch: bzr merge lp://qastaging/~tpeeters/ubuntu-ui-toolkit/09-desktop-toolbar
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Zsombor Egri Approve
Review via email: mp+213645@code.qastaging.launchpad.net

This proposal supersedes a proposal from 2014-04-01.

Commit message

Improve toolbar handling on desktop by opening it on mouse hover.

Description of the change

Improve toolbar handling on desktop by opening it on mouse hover.

To post a comment you must log in.
952. By Tim Peeters

mark properties that will be removed as deprecated

953. By Tim Peeters

comment

954. By Tim Peeters

update components.api

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Tim Peeters (tpeeters) wrote :

Tested on mako with image 272:
- Manually tested gallery-app, system-settings, camera-app.
- AP tests for address-book-app, messaging-app OK.

gallery-app tests fail with
File "/home/phablet/autopilot/gallery_app/emulators/main_screen.py", line 9, in <module>
from toolbar import Toolbar
ImportError: No module named 'toolbar' http://paste.ubuntu.com/7190772/
see http://paste.ubuntu.com/7190772/
This seems like a problem in gallery-app. See elopio's fix: https://code.launchpad.net/~elopio/gallery-app/override_toolbar/+merge/213703

There was one failure with unity8: http://paste.ubuntu.com/7191141/

Still need to run UITK tests.

955. By Tim Peeters

merge trunk

Revision history for this message
Tim Peeters (tpeeters) wrote :

unity AP test results:

Ran 38 tests in 1417.559s
OK
Restoring shell
unity8 start/running, process 11981

UITK AP test results:

Ran 148 tests in 939.315s
OK

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
956. By Tim Peeters

kick jenkins

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
957. By Tim Peeters

re-kick

Revision history for this message
Zsombor Egri (zsombi) wrote :

Good stuff ;)

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: