Thanks Simon.<br>I agree, is better keep the button class because of compatibility.<br><br>Reviewed-by Gonzalo Odiard <<a href="mailto:gonzalo@laptop.org">gonzalo@laptop.org</a>><br><br><br><br><div class="gmail_quote">
On Wed, Jul 27, 2011 at 11:58 AM, Simon Schampijer <span dir="ltr"><<a href="mailto:simon@schampijer.de">simon@schampijer.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">The keep button has led to a false assumption: learners<br>
confused it with a traditional save button and not recognising<br>
it as what it is, an option to duplicate an entry.<br>
<br>
Eben suggested to remove the button [1] and others have since<br>
then. The functionality of duplicating an entry will be moved<br>
to the entry palette in the Journal and the entry detail view [2].<br>
<br>
</div>We will print a warning when the KeepButton is used and remove<br>
it after another cycle.<br>
<div class="im"><br>
[1] <a href="http://lists.sugarlabs.org/archive/sugar-devel/2010-April/023439.html" target="_blank">http://lists.sugarlabs.org/archive/sugar-devel/2010-April/023439.html</a><br>
[2] <a href="http://lists.sugarlabs.org/archive/sugar-devel/2011-May/031316.html" target="_blank">http://lists.sugarlabs.org/archive/sugar-devel/2011-May/031316.html</a><br>
<br>
</div>Signed-off-by: Simon Schampijer <<a href="mailto:simon@laptop.org">simon@laptop.org</a>><br>
---<br>
 src/sugar/activity/widgets.py |   12 ++++++------<br>
 1 files changed, 6 insertions(+), 6 deletions(-)<br>
<div class="im"><br>
diff --git a/src/sugar/activity/widgets.py b/src/sugar/activity/widgets.py<br>
</div>index 9503d36..0c34a1f 100644<br>
<div class="im">--- a/src/sugar/activity/widgets.py<br>
+++ b/src/sugar/activity/widgets.py<br>
</div>@@ -18,6 +18,7 @@<br>
 import gtk<br>
 import gettext<br>
 import gconf<br>
+import logging<br>
<br>
 from sugar.graphics.toolbutton import ToolButton<br>
 from sugar.graphics.toolbarbox import ToolbarButton<br>
@@ -161,10 +162,13 @@ class ShareButton(RadioMenuButton):<br>
<div class="im">             self.neighborhood.handler_unblock(self._neighborhood_handle)<br>
<br>
<br>
</div>+# DEPRECATED<br>
<div class="im"> class KeepButton(ToolButton):<br>
<br>
     def __init__(self, activity, **kwargs):<br>
         ToolButton.__init__(self, **kwargs)<br>
</div>+        logging.warning('KeepButton has been deprecated since Sugar 0.94'<br>
+                        ' and should not be used in newly written code.')<br>
<div class="im">         self.props.tooltip = _('Keep')<br>
         self.props.accelerator = '<Ctrl>S'<br>
<br>
</div>@@ -232,8 +236,8 @@ class TitleEntry(gtk.ToolItem):<br>
<div class="im"><br>
<br>
 class ActivityToolbar(gtk.Toolbar):<br>
-    """The Activity toolbar with the Journal entry title, sharing,<br>
-       Keep and Stop buttons<br>
+    """The Activity toolbar with the Journal entry title, sharing<br>
+       and Stop buttons<br>
<br>
     All activities should have this toolbar. It is easiest to add it to your<br>
     Activity by using the ActivityToolbox.<br>
</div>@@ -261,10 +265,6 @@ class ActivityToolbar(gtk.Toolbar):<br>
<div><div></div><div class="h5">         self.share.show()<br>
         self.insert(self.share, -1)<br>
<br>
-        self.keep = KeepButton(activity)<br>
-        self.insert(self.keep, -1)<br>
-        self.keep.show()<br>
-<br>
         self.stop = StopButton(activity)<br>
         self.insert(self.stop, -1)<br>
         self.stop.show()<br>
--<br>
1.7.4.4<br>
<br>
_______________________________________________<br>
Sugar-devel mailing list<br>
<a href="mailto:Sugar-devel@lists.sugarlabs.org">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>
</div></div></blockquote></div><br>