Subject: [sup-talk] [PATCH] Show drafts in red, in thread-index-mode.

From: [email-masked] (Nicolas Pouillard)
Date: Fri, 18 Jan 2008 10:59:59 +0100

This patch would help to think about finishing drafts.
---
 bin/sup                            |    2 ++
 lib/sup/modes/thread-index-mode.rb |    5 +++--
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/bin/sup b/bin/sup
index 7314227..a49e2c4 100644
--- a/bin/sup
+++ b/bin/sup
@@ -137,6 +137,8 @@ begin
            Ncurses::A_BOLD
     c.add :index_starred_color, Ncurses::COLOR_YELLOW, Ncurses::COLOR_BLACK, 
            Ncurses::A_BOLD
+    c.add :index_draft_color, Ncurses::COLOR_RED, Ncurses::COLOR_BLACK, 
+           Ncurses::A_BOLD
     c.add :labellist_old_color, Ncurses::COLOR_WHITE, Ncurses::COLOR_BLACK
     c.add :labellist_new_color, Ncurses::COLOR_WHITE, Ncurses::COLOR_BLACK, 
            Ncurses::A_BOLD
diff --git a/lib/sup/modes/thread-index-mode.rb b/lib/sup/modes/thread-index-mode.rb
index 0d90be4..fac7500 100644
--- a/lib/sup/modes/thread-index-mode.rb
+++ b/lib/sup/modes/thread-index-mode.rb
@@ -599,7 +599,6 @@ protected
 
     date = t.date.to_nice_s
 
-    new = t.has_label?(:unread)
     starred = t.has_label?(:starred)
 
     ## format the from column
@@ -636,7 +635,9 @@ protected
     p = dp || t.participants.any? { |p| AccountManager.is_account? p }
 
     subj_color =
-      if new
+      if t.has_label?(:draft)
+        :index_draft_color
+      elsif t.has_label?(:unread)
         :index_new_color
       elsif starred
         :index_starred_color
-- 
1.5.3.1.109.gacd69

---

From: [email-masked] (Nicolas Pouillard)
Date: Fri, 18 Jan 2008 11:03:21 +0100

> This patch would help to think about finishing drafts.

Colors  and  UI are rather user dependent, so it's just in case...

-- 
Nicolas Pouillard aka Ertai

---

From: [email-masked] (William Morgan)
Date: Wed, 23 Jan 2008 10:22:11 -0800

> This patch would help to think about finishing drafts.

I like it. There's trailing whitespace on the c.add line. After that I'm
happy to apply it.

-- 
William <wmorgan-sup at masanjin.net>

---

From: [email-masked] (Nicolas Pouillard)
Date: Sat, 9 Feb 2008 13:05:29 +0100

This patch would help to think about finishing drafts.
---
 bin/sup                            |    2 ++
 lib/sup/modes/inbox-mode.rb        |    2 +-
 lib/sup/modes/thread-index-mode.rb |    5 +++--
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/bin/sup b/bin/sup
index 84fd77c..f536da6 100644
--- a/bin/sup
+++ b/bin/sup
@@ -138,6 +138,8 @@ begin
            Ncurses::A_BOLD
     c.add :index_starred_color, Ncurses::COLOR_YELLOW, Ncurses::COLOR_BLACK, 
            Ncurses::A_BOLD
+    c.add :index_draft_color, Ncurses::COLOR_RED, Ncurses::COLOR_BLACK,
+           Ncurses::A_BOLD
     c.add :labellist_old_color, Ncurses::COLOR_WHITE, Ncurses::COLOR_BLACK
     c.add :labellist_new_color, Ncurses::COLOR_WHITE, Ncurses::COLOR_BLACK, 
            Ncurses::A_BOLD
diff --git a/lib/sup/modes/inbox-mode.rb b/lib/sup/modes/inbox-mode.rb
index a2de13f..f156b69 100644
--- a/lib/sup/modes/inbox-mode.rb
+++ b/lib/sup/modes/inbox-mode.rb
@@ -9,7 +9,7 @@ class InboxMode < ThreadIndexMode
   end
 
   def initialize
-    super [:inbox, :sent], { :label => :inbox, :skip_killed => true }
+    super [:inbox, :sent, :draft], { :label => :inbox, :skip_killed => true }
     raise "can't have more than one!" if defined? @@instance
     @@instance = self
   end
diff --git a/lib/sup/modes/thread-index-mode.rb b/lib/sup/modes/thread-index-mode.rb
index e24d7e0..7a0b815 100644
--- a/lib/sup/modes/thread-index-mode.rb
+++ b/lib/sup/modes/thread-index-mode.rb
@@ -655,7 +655,6 @@ protected
 
     date = t.date.to_nice_s

     ## format the from column
@@ -692,7 +691,9 @@ protected
     p = dp || t.participants.any? { |p| AccountManager.is_account? p }
 
     subj_color =
-      if new

---

From: [email-masked] (William Morgan)
Date: Sat, 16 Feb 2008 11:20:07 -0800

> This patch would help to think about finishing drafts.

Merged into next. How do people like it? Thanks, Nicolas.