Archive of RubyForge sup-devel mailing list
 help / color / mirror / Atom feed
* [sup-devel] [PATCH] Bugfix: Set background color for starred messages
@ 2010-10-12 14:34 Gregor Hoffleit
  2010-10-14  1:10 ` Rich Lane
  0 siblings, 1 reply; 2+ messages in thread
From: Gregor Hoffleit @ 2010-10-12 14:34 UTC (permalink / raw)
  To: sup-devel

[-- Attachment #1: Type: text/plain, Size: 465 bytes --]

From the commit message:

    For the sake of consistency, Sup should always use the starred_color
    for the 'starred' column in the thread-index-mode, whether there's a
    star or a blank.

    Rationale: With this patch, I'm able to set the background color of
    the thread-index to bg:7 without having to change my term's
    background color.  Without the patch, the background color shines
    through in the starred column.

Regards,
    Gregor Hoffleit

[-- Attachment #2: 0001-Bugfix-Background-for-starred-messages.patch --]
[-- Type: application/octet-stream, Size: 1178 bytes --]

From 6551fc3e5373cc9f3a32b28b4335a2de2cd2efa4 Mon Sep 17 00:00:00 2001
From: Gregor Hoffleit <gregor@hoffleit.de>
Date: Thu, 27 May 2010 11:40:13 +0200
Subject: [PATCH] Bugfix: Set background color for starred column

For the sake of consistency, Sup should always use the starred_color for the
'starred' column in the thread-index-mode, whether there's a star or a blank.

Rationale: With this patch, I'm able to set the background color of the
thread-index to bg:7 without having to change my term's background color.
Without the patch, the background color shines through in the starred column.
---
 lib/sup/modes/thread-index-mode.rb |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/lib/sup/modes/thread-index-mode.rb b/lib/sup/modes/thread-index-mode.rb
index 5f3499c..f012123 100644
--- a/lib/sup/modes/thread-index-mode.rb
+++ b/lib/sup/modes/thread-index-mode.rb
@@ -887,7 +887,7 @@ protected
     [ 
       [:tagged_color, @tags.tagged?(t) ? ">" : " "],
       [:date_color, date_widget_text],
-      (starred ? [:starred_color, "*"] : [:none, " "]),
+      [:starred_color, (starred ? "*" : " ")],
     ] +
       from +
       [
-- 
1.5.6.5


[-- Attachment #3: Type: text/plain, Size: 143 bytes --]

_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [sup-devel] [PATCH] Bugfix: Set background color for starred messages
  2010-10-12 14:34 [sup-devel] [PATCH] Bugfix: Set background color for starred messages Gregor Hoffleit
@ 2010-10-14  1:10 ` Rich Lane
  0 siblings, 0 replies; 2+ messages in thread
From: Rich Lane @ 2010-10-14  1:10 UTC (permalink / raw)
  To: Gregor Hoffleit; +Cc: sup-devel

Applied to master.
_______________________________________________
Sup-devel mailing list
Sup-devel@rubyforge.org
http://rubyforge.org/mailman/listinfo/sup-devel


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2010-10-14  1:10 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2010-10-12 14:34 [sup-devel] [PATCH] Bugfix: Set background color for starred messages Gregor Hoffleit
2010-10-14  1:10 ` Rich Lane

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox