Merge lp://qastaging/~3v1n0/marlin/unity-launcher-support into lp://qastaging/marlin/0.x

Proposed by Marco Trevisan (Treviño)
Status: Merged
Merged at revision: 616
Proposed branch: lp://qastaging/~3v1n0/marlin/unity-launcher-support
Merge into: lp://qastaging/marlin/0.x
Diff against target: 420 lines (+319/-0)
4 files modified
libcore/marlin-progress-info.c (+40/-0)
libcore/marlin-progress-info.h (+2/-0)
src/CMakeLists.txt (+12/-0)
src/marlin-progress-ui-handler.c (+265/-0)
To merge this branch: bzr merge lp://qastaging/~3v1n0/marlin/unity-launcher-support
Reviewer Review Type Date Requested Status
xapantu Approve
ammonkey Pending
Review via email: mp+81864@code.qastaging.launchpad.net

Description of the change

Added support for the unity launcher icon to show the transfers status through a progress indicator, and to manage them using quicklists.

This is basically the port of what I already did for nautilus in [1].

You can see an example of how it works in this screencast (not so updated, by the way): http://www.youtube.com/watch?v=kpXlKM-9mIA

If you're interested I can also add an indicator support porting the patch [2].

[1] https://code.launchpad.net/~3v1n0/nautilus/ubuntu-unity-launcher-support/+merge/74401
[2] https://code.launchpad.net/~3v1n0/ubuntu/oneiric/nautilus/progress-indicator-support/+merge/70645

To post a comment you must log in.
Revision history for this message
xapantu (xapantu) wrote :

It works nice here, even with some canceled operations/multiple transfers, it's perfect! Thanks :)

review: Approve
Revision history for this message
xapantu (xapantu) wrote :

> If you're interested I can also add an indicator support porting the patch [2].
Oh, yes, I forgot to reply about that, but, of course, we are interested in an indicator support patch :)

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Ok, I'll push it as soon as I've time for it :)

However, could you add also the support for this on the elementary ppa?
It needs the libunity dependency, but most of users should have it.

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: