Merge lp:~nataliabidart/magicicada-gui/the-one-percent into lp:magicicada-gui

Proposed by Natalia Bidart
Status: Merged
Approved by: Natalia Bidart
Approved revision: 136
Merged at revision: 134
Proposed branch: lp:~nataliabidart/magicicada-gui/the-one-percent
Merge into: lp:magicicada-gui
Diff against target: 67 lines (+13/-6)
2 files modified
data/ui/operations.ui (+4/-3)
magicicada/gui/gtk/operations.py (+9/-3)
To merge this branch: bzr merge lp:~nataliabidart/magicicada-gui/the-one-percent
Reviewer Review Type Date Requested Status
Facundo Batista Approve
Review via email: mp+122886@code.launchpad.net

Commit message

- Added a legend to transfer progress bar indicating how much bytes has been sent
  (LP: #1043802).

To post a comment you must log in.
Revision history for this message
Facundo Batista (facundo) :
review: Approve
Revision history for this message
Natalia Bidart (nataliabidart) wrote :

There are additional revisions which have not been approved in review. Please seek review and approval of these new revisions.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'data/ui/operations.ui'
2--- data/ui/operations.ui 2012-08-21 20:10:09 +0000
3+++ data/ui/operations.ui 2012-09-06 19:38:19 +0000
4@@ -28,6 +28,8 @@
5 <column type="gint"/>
6 <!-- column-name show_transfer -->
7 <column type="gboolean"/>
8+ <!-- column-name transfer_text -->
9+ <column type="gchararray"/>
10 </columns>
11 </object>
12 <object class="GtkVBox" id="itself">
13@@ -79,11 +81,10 @@
14 <property name="alignment">1</property>
15 <signal name="clicked" handler="on_clear_button_clicked" swapped="no"/>
16 <child>
17- <object class="GtkCellRendererProgress" id="cellrendererprogress1">
18- <property name="width">100</property>
19- </object>
20+ <object class="GtkCellRendererProgress" id="cellrendererprogress1"/>
21 <attributes>
22 <attribute name="visible">6</attribute>
23+ <attribute name="text">7</attribute>
24 <attribute name="value">5</attribute>
25 </attributes>
26 </child>
27
28=== modified file 'magicicada/gui/gtk/operations.py'
29--- magicicada/gui/gtk/operations.py 2012-09-05 16:09:39 +0000
30+++ magicicada/gui/gtk/operations.py 2012-09-06 19:38:19 +0000
31@@ -26,6 +26,7 @@
32 # pylint: disable=E0611
33 from gi.repository import Gtk
34 # pylint: enable=E0611
35+from ubuntuone.controlpanel.gui import humanize
36
37 from magicicada import queue_content, syncdaemon
38 from magicicada.gui.gtk.helpers import Buildable
39@@ -43,6 +44,7 @@
40 REMOTE_ICON_NAME = u'folder-remote'
41 TIME_UNITS = {0: _(u'second'), 1: _(u'minute'), 2: _(u'hour'), 3: _(u'day')}
42 TRANSFER_OPS = (u'Upload', u'Download')
43+TRANSFER_TEXT = _('{transfered} of {total} ({percent:.1f}%)')
44
45
46 logger = logging.getLogger('magicicada.gui.gtk.operations')
47@@ -132,7 +134,7 @@
48 all_done = all(op[2][queue_content.DONE] for op in transfer_ops)
49 show_transfer = not all_done
50
51- row = row + (0, show_transfer)
52+ row = row + (0, show_transfer, '')
53 tree_iter = self.ops_store.append(parent_iter, row)
54
55 if row_key is not None:
56@@ -217,5 +219,9 @@
57 logger.exception('on_transfers: failed to retrive node from '
58 'local index to process transfer progress:')
59 else:
60- transfer = int((transfered / float(total)) * 100)
61- self.ops_store.set_value(row_iter, 5, transfer)
62+ transfer = (transfered / float(total)) * 100
63+ self.ops_store.set_value(row_iter, 5, int(transfer))
64+ text = TRANSFER_TEXT.format(transfered=humanize(transfered),
65+ total=humanize(total),
66+ percent=transfer)
67+ self.ops_store.set_value(row_iter, 7, text)

Subscribers

People subscribed via source and target branches

to all changes: