Ticket #1262: sugar-1262.patch

src/jarabe/journal/expandedentry.py

@@ -159 +159 @@
         return icon
 
     def _create_title(self):
-        title = CanvasEntry()
-        title.set_background(style.COLOR_WHITE.get_html())
-        title.props.text = self._metadata.get('title', _('Untitled'))
-        title.props.widget.connect('focus-out-event',
-                                   self._title_focus_out_event_cb)
-        return title
+        entry = gtk.Entry()
+        entry.props.text = self._metadata.get('title', _('Untitled'))
+
+        bg_color = style.COLOR_WHITE.get_gdk_color()
+        entry.modify_bg(gtk.STATE_INSENSITIVE, bg_color)
+        entry.modify_base(gtk.STATE_INSENSITIVE, bg_color)
+
+        entry.props.editable = model.is_editable(self._metadata)
+        if entry.props.editable:
+            entry.connect('focus-out-event', self._title_focus_out_event_cb)
+
+        return hippo.CanvasWidget(widget=entry)
 
     def _create_date(self):
         date = hippo.CanvasText(xalign=hippo.ALIGNMENT_START,
@@ -297 +303 @@
         vbox.append(text_view, hippo.PACK_EXPAND)
 
         text_view.text_view_widget.props.accepts_tab = False
-        text_view.text_view_widget.connect('focus-out-event',
-                                           self._description_focus_out_event_cb)
+        editable = model.is_editable(self._metadata)
+        if editable:
+            text_view.text_view_widget.connect('focus-out-event',
+                    self._description_focus_out_event_cb)
+        text_view.text_view_widget.props.editable = editable
 
         return vbox, text_view
 
@@ -322 +331 @@
         vbox.append(text_view, hippo.PACK_EXPAND)
 
         text_view.text_view_widget.props.accepts_tab = False
-        text_view.text_view_widget.connect('focus-out-event',
-                                           self._tags_focus_out_event_cb)
+        editable = model.is_editable(self._metadata)
+        if editable:
+            text_view.text_view_widget.connect('focus-out-event',
+                    self._tags_focus_out_event_cb)
+        text_view.text_view_widget.props.editable = editable
 
         return vbox, text_view
 
@@ -345 +357 @@
         needs_update = False
 
         old_title = self._metadata.get('title', None)
-        if old_title != self._title.props.text:
-            self._icon.palette.props.primary_text = self._title.props.text
-            self._metadata['title'] = self._title.props.text
+        new_title = self._title.props.widget.props.text
+        if old_title != new_title:
+            self._icon.palette.props.primary_text = new_title
+            self._metadata['title'] = new_title
             self._metadata['title_set_by_user'] = '1'
             needs_update = True
 
@@ -373 +386 @@
         return int(self._metadata.get('keep', 0)) == 1
 
     def _keep_icon_activated_cb(self, keep_icon):
+        if not model.is_editable(self._metadata):
+            return
         if self.get_keep():
             self._metadata['keep'] = 0
         else:

src/jarabe/journal/listview.py

@@ -261 +261 @@
     def __favorite_clicked_cb(self, cell, path):
         row = self._model[path]
         metadata = model.get(row[ListModel.COLUMN_UID])
+        if not model.is_editable(metadata):
+            return
         if metadata.get('keep', 0) == '1':
             metadata['keep'] = '0'
         else:
@@ -310 +312 @@
         else:
             self._clear_message()
 
+        is_editable = False
+        mountpoints = self._query.get('mountpoints')
+        if mountpoints is not None and len(mountpoints):
+            is_editable = mountpoints[0] == '/'
+        self.cell_title.props.editable = is_editable
+
     def _is_query_empty(self):
         # FIXME: This is a hack, we shouldn't have to update this every time
         # a new search term is added.

src/jarabe/journal/model.py

@@ -371 +371 @@
 def _datastore_deleted_cb(object_id):
     deleted.send(None, object_id=object_id)
 
-def find(query, page_size):
+def find(query_, page_size):
     """Returns a ResultSet
     """
+    query = query_.copy()
+
     if 'order_by' not in query:
         query['order_by'] = ['-mtime']
 
@@ -529 +531 @@
 
     return file_name
 
+def is_editable(metadata):
+    mountpoint = metadata.get('mountpoint', '/')
+    return mountpoint == '/'
+
 created = dispatch.Signal()
 updated = dispatch.Signal()
 deleted = dispatch.Signal()
-