Subject: [sup-talk] [PATCH] Put labels before subject in thread index view.

From: [email-masked] (Mark Alexander)
Date: Sat, 23 May 2009 11:25:57 -0700

This patch is probably controversial, and I expect it
to be rejected.  But I really like the way Gmail puts
the labels before the subject, and I've duplicated that here.
It helps out at work, where subject lines tend to be very
long, pushing the labels past the right edge of the window.
---
 lib/sup/modes/thread-index-mode.rb |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/lib/sup/modes/thread-index-mode.rb b/lib/sup/modes/thread-index-mode.rb
index f65d241..897dca7 100644
--- a/lib/sup/modes/thread-index-mode.rb
+++ b/lib/sup/modes/thread-index-mode.rb
@@ -843,10 +843,11 @@ protected
       [subj_color, size_widget_text],
       [:to_me_color, t.labels.member?(:attachment) ? "@" : " "],
       [:to_me_color, dp ? ">" : (p ? '+' : " ")],
-      [subj_color, t.subj + (t.subj.empty? ? "" : " ")],
     ] +
-      (t.labels - @hidden_labels).map { |label| [:label_color, "+#{label} "] } +
-      [[:snippet_color, snippet]
+      (t.labels - @hidden_labels).map { |label| [:label_color, "#{label} "] } +
+      [
+      [subj_color, t.subj + (t.subj.empty? ? "" : " ")],
+      [:snippet_color, snippet],
     ]
   end
 
-- 
1.5.6.3

---

From: [email-masked] (Ben Walton)
Date: Sat, 23 May 2009 15:22:28 -0400

> long, pushing the labels past the right edge of the window.

I'll give the idea a +1.  I'd prefer labels first also.  It would also
be more uniform when you get mail without a subject...[why yes, I do
get those from people! :(].

I didn't look at the patch itself, so this comment is purely on the
idea.

-Ben

-- 
Ben Walton
Systems Programmer - CHASS
University of Toronto
C:416.407.5610 | W:416.978.4302

GPG Key Id: 8E89F6D2; Key Server: pgp.mit.edu
Contact me to arrange for a CAcert assurance meeting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://rubyforge.org/pipermail/sup-talk/attachments/20090523/df5d8b16/attachment.bin>

---

From: [email-masked] (Nicolas Pouillard)
Date: Sat, 23 May 2009 23:42:29 +0200

> I didn't look at the patch itself, so this comment is purely on the

+1 to the idea too!

-- 
Nicolas Pouillard

---

From: [email-masked] (Henri Ducrocq)
Date: Sun, 24 May 2009 13:48:41 +0100

Excerpts from Marc Hartstein's message of Sun May 24 01:42:20 +0100 2009:
> This is a great idea, but can we get it configurable/delegated to a
> hook?  Bonus points if there's a straightforward way to be able to
> switch between the two modes at runtime.
> 
> I've occasionally thought I'd want this, as I get a lot of long subject
> too, but I wouldn't want multiply-tagged messages [especially with
> special tags like inbox and unread] to have the tags obscure the
> subject.

We could have an even more configurable presentation of the messages,
with the possibility of having a multiline view, e.g. subject on the
first line, labels on the second, excerpt of the text on the 3rd...

-- 
Henri

---

From: [email-masked] (William Morgan)
Date: Wed, 27 May 2009 09:07:08 -0700

Reformatted excerpts from Marc Hartstein's message of 2009-05-23:
> hook?

I'd be happy to add an index-model-subject-widget hook, similar to the
index-mode-size-widget hook, if someone wants to write one.

> I've occasionally thought I'd want this, as I get a lot of long
> subject too, but I wouldn't want multiply-tagged messages [especially
> with special tags like inbox and unread] to have the tags obscure the
> subject.

Unread, starred, and other system labels shouldn't be displayed in
thread-view-mode. Index will, except in inbox-mode. It might not be too
bad. 

I'll apply this patch and we'll see who yells.
-- 
William <wmorgan-sup at masanjin.net>