[Sugar-devel] [PATCH 2/2] Backport of "Remove the Naming Alert" in sugar-toolkit-gtk3
godiard at sugarlabs.org
godiard at sugarlabs.org
Thu Apr 19 15:22:22 EDT 2012
From: Gonzalo Odiard <godiard at gmail.com>
With the new 'Write to Journal anytime' feature it is possible
to write a description within an activity. Hopefully this leads
to more usage of the Journal for reflection by the learner. The
hinting nature ('you have not named your session yet') of the
Naming Alert has not been replaced, though.
This feature is requested by teachers and will provide a more consistent
experience in sugar 0.96 between gtk2 and gtk3 activities.
The original patch is from Simon Schampijer <simon at laptop.org>
Signed-off-by: Gonzalo Odiard <gonzalo at laptop.org>
---
po/POTFILES.in | 1 -
src/sugar/activity/Makefile.am | 1 -
src/sugar/activity/activity.py | 18 +--
src/sugar/activity/namingalert.py | 306 -------------------------------------
4 files changed, 5 insertions(+), 321 deletions(-)
delete mode 100644 src/sugar/activity/namingalert.py
diff --git a/po/POTFILES.in b/po/POTFILES.in
index c9a8443..f3c984e 100644
--- a/po/POTFILES.in
+++ b/po/POTFILES.in
@@ -1,5 +1,4 @@
src/sugar/activity/activity.py
-src/sugar/activity/namingalert.py
src/sugar/activity/widgets.py
src/sugar/graphics/alert.py
src/sugar/graphics/colorbutton.py
diff --git a/src/sugar/activity/Makefile.am b/src/sugar/activity/Makefile.am
index f3a663a..ca94b4d 100644
--- a/src/sugar/activity/Makefile.am
+++ b/src/sugar/activity/Makefile.am
@@ -7,5 +7,4 @@ sugar_PYTHON = \
activityservice.py \
bundlebuilder.py \
i18n.py \
- namingalert.py \
widgets.py
diff --git a/src/sugar/activity/activity.py b/src/sugar/activity/activity.py
index 5844506..fdf9827 100644
--- a/src/sugar/activity/activity.py
+++ b/src/sugar/activity/activity.py
@@ -75,7 +75,6 @@ from sugar import util
from sugar.presence import presenceservice
from sugar.activity import i18n
from sugar.activity.activityservice import ActivityService
-from sugar.activity.namingalert import NamingAlert
from sugar.graphics import style
from sugar.graphics.window import Window
from sugar.graphics.alert import Alert
@@ -900,19 +899,12 @@ class Activity(Window, gtk.Container):
self.emit('_closing')
- if skip_save or self._jobject is None or \
- self.metadata.get('title_set_by_user', '0') == '1':
- if not self._closing:
- if not self._prepare_close(skip_save):
- return
+ if not self._closing:
+ if not self._prepare_close(skip_save):
+ return
- if not self._updating_jobject:
- self._complete_close()
- else:
- title_alert = NamingAlert(self, get_bundle_path())
- title_alert.set_transient_for(self.get_toplevel())
- title_alert.show()
- self.reveal()
+ if not self._updating_jobject:
+ self._complete_close()
def __realize_cb(self, window):
wm.set_bundle_id(window.window, self.get_bundle_id())
diff --git a/src/sugar/activity/namingalert.py b/src/sugar/activity/namingalert.py
deleted file mode 100644
index 078c27a..0000000
--- a/src/sugar/activity/namingalert.py
+++ /dev/null
@@ -1,306 +0,0 @@
-# Copyright (C) 2009 One Laptop Per Child
-#
-# This library is free software; you can redistribute it and/or
-# modify it under the terms of the GNU Lesser General Public
-# License as published by the Free Software Foundation; either
-# version 2 of the License, or (at your option) any later version.
-#
-# This library is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
-# Lesser General Public License for more details.
-#
-# You should have received a copy of the GNU Lesser General Public
-# License along with this library; if not, write to the
-# Free Software Foundation, Inc., 59 Temple Place - Suite 330,
-# Boston, MA 02111-1307, USA.
-
-import gettext
-import os
-
-import gio
-import gtk
-import gobject
-import gconf
-
-from sugar.graphics import style
-from sugar.graphics.icon import Icon
-from sugar.graphics.xocolor import XoColor
-from sugar.graphics.icon import get_icon_file_name
-from sugar.graphics.toolbutton import ToolButton
-
-from sugar.bundle.activitybundle import ActivityBundle
-
-
-_ = lambda msg: gettext.dgettext('sugar-toolkit', msg)
-
-
-def _get_icon_name(metadata):
- file_name = None
-
- mime_type = metadata.get('mime_type', '')
- if not file_name and mime_type:
- icons = gio.content_type_get_icon(mime_type)
- for icon_name in icons.props.names:
- file_name = get_icon_file_name(icon_name)
- if file_name is not None:
- break
-
- if file_name is None or not os.path.exists(file_name):
- file_name = get_icon_file_name('application-octet-stream')
-
- return file_name
-
-
-class NamingToolbar(gtk.Toolbar):
- """ Toolbar of the naming alert
- """
-
- __gtype_name__ = 'SugarNamingToolbar'
-
- __gsignals__ = {
- 'keep-clicked': (gobject.SIGNAL_RUN_FIRST, gobject.TYPE_NONE, ([])),
- }
-
- def __init__(self):
- gtk.Toolbar.__init__(self)
-
- client = gconf.client_get_default()
- color = XoColor(client.get_string('/desktop/sugar/user/color'))
- icon = Icon()
- icon.set_from_icon_name('activity-journal',
- gtk.ICON_SIZE_LARGE_TOOLBAR)
- icon.props.xo_color = color
- self._add_widget(icon)
-
- self._add_separator()
-
- self._title = gtk.Label(_('Name this entry'))
- self._add_widget(self._title)
-
- self._add_separator(True)
-
- self._keep_button = ToolButton('dialog-ok', tooltip=_('Keep'))
- self._keep_button.props.accelerator = 'Return'
- self._keep_button.connect('clicked', self.__keep_button_clicked_cb)
- self.insert(self._keep_button, -1)
- self._keep_button.show()
-
- def _add_separator(self, expand=False):
- separator = gtk.SeparatorToolItem()
- separator.props.draw = False
- if expand:
- separator.set_expand(True)
- else:
- separator.set_size_request(style.DEFAULT_SPACING, -1)
- self.insert(separator, -1)
- separator.show()
-
- def _add_widget(self, widget, expand=False):
- tool_item = gtk.ToolItem()
- tool_item.set_expand(expand)
-
- tool_item.add(widget)
- widget.show()
-
- self.insert(tool_item, -1)
- tool_item.show()
-
- def __keep_button_clicked_cb(self, widget, data=None):
- self.emit('keep-clicked')
-
-
-class FavoriteIcon(gtk.ToggleButton):
-
- def __init__(self):
- gtk.ToggleButton.__init__(self)
- self.set_relief(gtk.RELIEF_NONE)
- self.set_focus_on_click(False)
-
- self._icon = Icon(icon_name='emblem-favorite',
- pixel_size=style.SMALL_ICON_SIZE)
- self.set_image(self._icon)
-
- self.connect('toggled', self.__toggled_cb)
- self.connect('leave-notify-event', self.__leave_notify_event_cb)
- self.connect('enter-notify-event', self.__enter_notify_event_cb)
-
- def __toggled_cb(self, widget):
- if self.get_active():
- client = gconf.client_get_default()
- color = XoColor(client.get_string('/desktop/sugar/user/color'))
- self._icon.props.xo_color = color
- else:
- self._icon.props.stroke_color = style.COLOR_BUTTON_GREY.get_svg()
- self._icon.props.fill_color = style.COLOR_WHITE.get_svg()
-
- def __enter_notify_event_cb(self, icon, event):
- if not self.get_active():
- self._icon.props.fill_color = style.COLOR_BUTTON_GREY.get_svg()
-
- def __leave_notify_event_cb(self, icon, event):
- if not self.get_active():
- self._icon.props.fill_color = style.COLOR_TRANSPARENT.get_svg()
-
-
-class NamingAlert(gtk.Window):
-
- __gtype_name__ = 'SugarNamingAlert'
-
- def __init__(self, activity, bundle_path):
- gtk.Window.__init__(self)
-
- self._bundle_path = bundle_path
- self._favorite_icon = None
- self._title = None
- self._description = None
- self._tags = None
-
- accel_group = gtk.AccelGroup()
- self.set_data('sugar-accel-group', accel_group)
- self.add_accel_group(accel_group)
-
- self.set_border_width(style.LINE_WIDTH)
- offset = style.GRID_CELL_SIZE
- width = gtk.gdk.screen_width() - offset * 2
- height = gtk.gdk.screen_height() - offset * 2
- self.set_size_request(width, height)
- self.set_position(gtk.WIN_POS_CENTER_ALWAYS)
- self.set_decorated(False)
- self.set_resizable(False)
- self.set_modal(True)
- self.connect('realize', self.__realize_cb)
-
- self._activity = activity
-
- vbox = gtk.VBox()
- self.add(vbox)
- vbox.show()
-
- toolbar = NamingToolbar()
- toolbar.connect('keep-clicked', self.__keep_cb)
- vbox.pack_start(toolbar, False)
- toolbar.show()
-
- body = self._create_body()
- vbox.pack_start(body, expand=True, fill=True)
- body.show()
-
- self._title.grab_focus()
-
- def _create_body(self):
- body = gtk.VBox(spacing=style.DEFAULT_SPACING)
- body.set_border_width(style.DEFAULT_SPACING * 3)
- header = self._create_header()
- body.pack_start(header, expand=False, padding=style.DEFAULT_PADDING)
-
- body.pack_start(self._create_separator(style.DEFAULT_SPACING),
- expand=False)
-
- body.pack_start(self._create_label(_('Description:')), expand=False)
-
- description = self._activity.metadata.get('description', '')
- description_box, self._description = self._create_text_view(description)
- body.pack_start(description_box, expand=True, fill=True)
-
- body.pack_start(self._create_separator(style.DEFAULT_PADDING),
- expand=False)
-
-
- body.pack_start(self._create_label(_('Tags:')), expand=False)
-
- tags = self._activity.metadata.get('tags', '')
- tags_box, self._tags = self._create_text_view(tags)
- body.pack_start(tags_box, expand=True, fill=True)
-
- body.show_all()
- return body
-
- def _create_label(self, text):
- text = gtk.Label(text)
- text.set_alignment(0, 0.5)
- text.modify_fg(gtk.STATE_NORMAL,
- style.COLOR_BUTTON_GREY.get_gdk_color())
- return text
-
- def _create_separator(self, height):
- separator = gtk.HSeparator()
- separator.modify_bg(gtk.STATE_NORMAL, style.COLOR_WHITE.get_gdk_color())
- separator.set_size_request(-1, height)
- return separator
-
- def _create_header(self):
- header = gtk.HBox(spacing=style.DEFAULT_SPACING)
-
- self._favorite_icon = FavoriteIcon()
- header.pack_start(self._favorite_icon, expand=False)
-
- entry_icon = self._create_entry_icon()
- header.pack_start(entry_icon, expand=False)
-
- self._title = self._create_title()
- header.pack_start(self._title, expand=True)
-
- return header
-
- def _create_entry_icon(self):
- bundle_id = self._activity.metadata.get('activity', '')
- if not bundle_id:
- bundle_id = self._activity.metadata.get('bundle_id', '')
-
- if bundle_id == '':
- file_name = _get_icon_name(self._activity.metadata)
- else:
- activity_bundle = ActivityBundle(self._bundle_path)
- file_name = activity_bundle.get_icon()
- entry_icon = Icon(file=file_name, icon_size=gtk.ICON_SIZE_LARGE_TOOLBAR)
- if self._activity.metadata.get('icon-color'):
- entry_icon.props.xo_color = XoColor( \
- self._activity.metadata['icon-color'])
- return entry_icon
-
- def _create_title(self):
- title = gtk.Entry()
- title.set_text(self._activity.metadata.get('title', _('Untitled')))
- return title
-
- def _create_text_view(self, text):
- scrolled_window = gtk.ScrolledWindow()
- scrolled_window.set_policy(gtk.POLICY_NEVER, gtk.POLICY_AUTOMATIC)
- scrolled_window.set_border_width(style.LINE_WIDTH)
- scrolled_window.set_shadow_type(gtk.SHADOW_IN)
-
- text_view = gtk.TextView()
- text_view.set_left_margin(style.DEFAULT_PADDING)
- text_view.set_wrap_mode(gtk.WRAP_WORD_CHAR)
- text_view.set_accepts_tab(False)
- text_view.get_buffer().set_text(text)
- scrolled_window.add(text_view)
-
- return scrolled_window, text_view
-
- def __realize_cb(self, widget):
- self.window.set_type_hint(gtk.gdk.WINDOW_TYPE_HINT_DIALOG)
- self.window.set_accept_focus(True)
-
- def __keep_cb(self, widget):
- if self._favorite_icon.get_active():
- self._activity.metadata['keep'] = 1
- else:
- self._activity.metadata['keep'] = 0
-
- self._activity.metadata['title'] = self._title.get_text()
-
- text_buffer = self._tags.get_buffer()
- start, end = text_buffer.get_bounds()
- new_tags = text_buffer.get_text(start, end)
- self._activity.metadata['tags'] = new_tags
-
- text_buffer = self._description.get_buffer()
- start, end = text_buffer.get_bounds()
- new_description = text_buffer.get_text(start, end)
- self._activity.metadata['description'] = new_description
-
- self._activity.metadata['title_set_by_user'] = '1'
- self._activity.close()
- self.destroy()
--
1.7.7.6
More information about the Sugar-devel
mailing list