Bug 56321

Summary:	More whitespace in read/unread mails column
Product:	[Applications] kmail	Reporter:	Lauri Watts <lauri>
Component:	general	Assignee:	kdepim bugs <kdepim-bugs>
Status:	RESOLVED FIXED
Severity:	wishlist
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	Compiled Sources
OS:	Linux
Latest Commit:		Version Fixed In:

Description Lauri Watts 2003-03-24 15:55:36 UTC

Version:            (using KDE Devel)
Installed from:    Compiled sources

If the 'unread' and 'total' columns are shown in the folder pane in kmail, the contents are right aligned.  This leaves no white space between the text and the splitter, which makes it both unattractive and difficult to read (I keep thinking that I need to scroll to the right too see the rest of the column)

Possible solutions:
1: KNode has the columns centered under their heading, which is much easier to read, and takes up no more space, since the headings are wider than the contents in any case, unless you have 6 figures worth of unread mail.

2: Force a few pixels (2-5 would probably be enough to make this look a whole lot better) between the end of the text in the columns and the splitter.

Comment 1 Ingo Kl 2003-03-24 23:00:58 UTC

Subject: KDE_3_1_BRANCH: kdenetwork/libkdenetwork

CVS commit by kloecker: 

Add a space after the number of unread/total messages.

CCMAIL:56321-fixed@bugs.kde.org


  M +12 -4     kfoldertree.cpp   1.23.2.4


--- kdenetwork/libkdenetwork/kfoldertree.cpp  #1.23.2.3:1.23.2.4
@@ -103,6 +103,10 @@ void KFolderTreeItem::setUnreadCount( in
   
   QString unread = QString::null;
-  if (mUnread == 0) unread = "-";
-  else unread.setNum(mUnread);
+  if (mUnread == 0)
+    unread = "- ";
+  else {
+    unread.setNum(mUnread);
+    unread += " ";
+  }
 
   setText( static_cast<KFolderTree*>(listView())->unreadIndex(), 
@@ -118,6 +122,10 @@ void KFolderTreeItem::setTotalCount( int
 
   QString total = QString::null;
-  if (mTotal == 0) total = "-";
-  else total.setNum(mTotal);
+  if (mTotal == 0)
+    total = "- ";
+  else {
+    total.setNum(mTotal);
+    total += " ";
+  }
 
   setText( static_cast<KFolderTree*>(listView())->totalIndex(),