[Bps-public-commit] App-Todo branch, master, updated. 0.97-2-g284859b

[Thomas Sibley]

The branch, master has been updated
       via  284859b35f4f8b43d20e6ebd299d271d5a4a5799 (commit)
      from  84883e745b23520038afc8c422f19c010a0d7229 (commit)

Summary of changes:
 bin/todo.pl |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

- Log -----------------------------------------------------------------
commit 284859b35f4f8b43d20e6ebd299d271d5a4a5799
Author: Thomas Sibley <trs at bestpractical.com>
Date:   Sat Oct 30 10:03:06 2010 -0400

    Be more nitpicky about printing spaces with a "color" that might show up
    
    In some cases color('dark') is underlined by the terminal, and underlined
    spaces between parts of the task metadata look bad.

diff --git a/bin/todo.pl b/bin/todo.pl
index a78d00d..24d189b 100755
--- a/bin/todo.pl
+++ b/bin/todo.pl
@@ -289,9 +289,9 @@ sub list_tasks {
                       (overdue($t->{due}) ? 'magenta' : 'dark'))
             if $t->{due};
 
-        print color('dark');
         if ($t->{tags}) {
-            print ' [' . $t->{tags} . ']';
+            print color('reset'), ' ', color('dark');
+            print '[' . $t->{tags} . ']';
         }
 
         $t->{owner} =~ s/<nobody>/<nobody\@localhost>/;
@@ -303,7 +303,8 @@ sub list_tasks {
         my $not_owner = lc $owner->address ne lc $config{email};
         my $not_requestor = lc $requestor->address ne lc $config{email};
         if( $t->{group} || $not_owner || $not_requestor ) {
-            print ' (';
+            print color('reset'), ' ', color('dark');
+            print '(';
             print join(", ",
                        $t->{group} || "personal",
                        $not_requestor ? "for " . $requestor->name : (),

-----------------------------------------------------------------------