Ticket #37: posterity.diff

trunk/posterity/model.py

@@ -288 +288 @@
             msg.is_deleted = False
         self._update_date_cache()
 
-    def mark_as_read(self, deleted=False):
-        """Mark all messages in the conversation as read
+    def mark_as_read(self, is_read=True, deleted=False):
+        """Mark all messages in the conversation as `is_read`
         
         If @deleted is True all messages marked for deletions are marked
-        as read.
+        as `is_read`.
         """
         if deleted:
             for msg in self.deleted_msgs:
-                msg.is_unread = False
+                msg.is_unread = not is_read
         else:
             for msg in self.msgs:
-                msg.is_unread = False
+                msg.is_unread = not is_read
             
     def _update_date_cache(self):
         """

trunk/posterity/controllers.py

@@ -86 +86 @@
             elif do_action == 'undelete':
                 c.remove_tag('Trash')
                 c.undelete()
+            elif do_action == 'unread':
+                c.mark_as_read(is_read=False)
             elif do_action.startswith('tag_'):
                 t = do_action[4:]
                 c.add_tag(t)

trunk/posterity/templates/conv_index.html

@@ -34 +34 @@
 
     <select py:if="account.tags"
             onchange="submitAction(this.options[this.selectedIndex].value)">
-      <option>Add/Remove Tag</option>
+      <option>More actions...</option>
+      <option value="unread">Mark as unread</option>
       <optgroup label="Add Tag">
         <option py:for="t in account.tags" py:if="not t.is_special"
                 value="tag_$t.name">$t.name</option>

trunk/posterity/templates/conv_show.html

@@ -46 +46 @@
             onclick="submitAction('undelete')">Undelete</button>
     <select py:if="account.tags"
             onchange="submitAction(this.options[this.selectedIndex].value)">
-      <option>Add/Remove Tag</option>
+      <option>More actions...</option>
+      <option value="unread">Mark as unread</option>
       <optgroup label="Add Tag">
         <option py:for="t in account.tags" py:if="not t.is_special"
                 value="tag_$t.name">$t.name</option>