<br><br><div class="gmail_quote">On Tue, Jun 7, 2011 at 11:51 AM, Rafael Ortiz <span dir="ltr"><<a href="mailto:rafael@activitycentral.com">rafael@activitycentral.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5"><div class="gmail_quote">On Tue, Jun 7, 2011 at 11:31 AM, Sascha Silbe <span dir="ltr"><<a href="mailto:silbe@activitycentral.com" target="_blank">silbe@activitycentral.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The warning previously used the word "cancel" to describe two antagonistic<br>
operations: Not continuing the download (including erasing it from the<br>
Journal) and not stopping Browse.<br>
<br>
The new wording was agreed upon [1] by the Design Team.<br>
<br>
[1] <a href="http://meeting.sugarlabs.org/sugar-meeting/meetings/2011-05-29T16:09:15.html" target="_blank">http://meeting.sugarlabs.org/sugar-meeting/meetings/2011-05-29T16:09:15.html</a><br>
<br>
Closes: SL#1508<br>
Signed-off-by: Sascha Silbe <<a href="mailto:silbe@activitycentral.com" target="_blank">silbe@activitycentral.com</a>><br>
---<br>
downloadmanager.py | 4 ++++<br>
webactivity.py | 14 +++++++++++---<br>
2 files changed, 15 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/downloadmanager.py b/downloadmanager.py<br>
index a88389f..4eab726 100644<br>
--- a/downloadmanager.py<br>
+++ b/downloadmanager.py<br>
@@ -64,6 +64,10 @@ def can_quit():<br>
return len(_active_downloads) == 0<br>
<br>
<br>
+def num_downloads():<br>
+ return len(_active_downloads)<br>
+<br>
+<br>
def remove_all_downloads():<br>
for download in _active_downloads:<br>
download.cancelable.cancel(NS_ERROR_FAILURE)<br>
diff --git a/webactivity.py b/webactivity.py<br>
index 48cb3ed..fc9e913 100644<br>
--- a/webactivity.py<br>
+++ b/webactivity.py<br>
@@ -18,6 +18,7 @@<br>
<br>
import logging<br>
from gettext import gettext as _<br>
+from gettext import ngettext<br>
import os<br>
import subprocess<br>
<br>
@@ -650,10 +651,17 @@ class WebActivity(activity.Activity):<br>
return True<br>
else:<br>
alert = Alert()<br>
- alert.props.title = _('Download in progress')<br>
- alert.props.msg = _('Stopping now will cancel your download')<br>
+ alert.props.title = ngettext('Download in progress',<br>
+ 'Downloads in progress',<br>
+ downloadmanager.num_downloads())<br>
+ message = ngettext('Stopping now will erase your download',<br>
+ 'Stopping now will erase your downloads',<br>
+ downloadmanager.num_downloads())<br>
+ alert.props.msg = message<br>
cancel_icon = Icon(icon_name='dialog-cancel')<br>
- alert.add_button(gtk.RESPONSE_CANCEL, _('Cancel'), cancel_icon)<br>
+ cancel_label = ngettext('Continue download', 'Continue downloads',<br>
+ downloadmanager.num_downloads())<br>
+ alert.add_button(gtk.RESPONSE_CANCEL, cancel_label, cancel_icon)<br>
stop_icon = Icon(icon_name='dialog-ok')<br>
alert.add_button(gtk.RESPONSE_OK, _('Stop'), stop_icon)<br>
stop_icon.show()<br>
<font color="#888888">--<br>
1.7.4.1<br>
<br>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org" target="_blank">Sugar-devel@lists.sugarlabs.org</a><br>
<a href="http://lists.sugarlabs.org/listinfo/sugar-devel" target="_blank">http://lists.sugarlabs.org/listinfo/sugar-devel</a><br>
</font></blockquote></div><div><br></div></div></div><div>One comment here: when one stops a download described by </div><div>'Stopping now will erase your download', it</div><div>also quits the activity. </div><div>
<br></div>
<div>agree with the 'Continue download', dialog. </div><div><br></div><div><br></div>
</blockquote></div>We can push this change, it's better than the actual wording.<div><br></div><div><br></div>