[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kmail-devel
Subject:    patch for addressesdialog
From:       Derek Kite <dkite () shaw ! ca>
Date:       2003-06-05 3:08:56
[Download RAW message or body]

In the Address Selection dialog, this expands the Selected Addresses.

Currently, when an address is selected and moved to Selected Addresses, what 
you see is  

+To: . 

What this does is expands it so the addresses are visible. It's a usability 
issue, ie.

 -To
    -someone@somewhere.here
    -....

What is missing is when the field has an address when the dialog is opened. 
I'll try to track that down.

Derek

["addrssesdialog.diff" (text/x-diff)]

Index: addressesdialog.cpp
===================================================================
RCS file: /home/kde/kdepim/libkdepim/addressesdialog.cpp,v
retrieving revision 1.7
diff -u -3 -p -r1.7 addressesdialog.cpp
--- addressesdialog.cpp	29 May 2003 10:41:13 -0000	1.7
+++ addressesdialog.cpp	1 Jun 2003 03:38:48 -0000
@@ -371,8 +371,10 @@ AddressesDialog::addSelectedTo()
   for( KABC::Addressee::List::Iterator it = lst.begin(); it != lst.end(); ++it ) {
     addAddresseeToSelected( *it, d->toItem );
   }
-  if ( d->toItem->childCount() > 0 )
+  if ( d->toItem->childCount() > 0 ){
     d->toItem->setVisible( true );
+    d->toItem->setOpen( true );
+  }
 }
 
 void
@@ -385,8 +387,10 @@ AddressesDialog::addSelectedCC()
   for( KABC::Addressee::List::Iterator it = lst.begin(); it != lst.end(); ++it ) {
     addAddresseeToSelected( *it, d->ccItem );
   }
-  if ( d->ccItem->childCount() > 0 )
+  if ( d->ccItem->childCount() > 0 ){
     d->ccItem->setVisible( true );
+    d->ccItem->setOpen( true );
+  }
 }
 
 void
@@ -399,8 +403,10 @@ AddressesDialog::addSelectedBCC()
   for( KABC::Addressee::List::Iterator it = lst.begin(); it != lst.end(); ++it ) {
     addAddresseeToSelected( *it, d->bccItem );
   }
-  if ( d->bccItem->childCount() > 0 )
+  if ( d->bccItem->childCount() > 0 ){
     d->bccItem->setVisible( true );
+    d->bccItem->setOpen( true );
+  }
 }
 
 void


_______________________________________________
KMail Developers mailing list
kmail@mail.kde.org
http://mail.kde.org/mailman/listinfo/kmail


[prev in list] [next in list] [prev in thread] [next in thread]