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

List:       kmail-devel
Subject:    Re: Implemented a new feature for KMail, please review
From:       "Aaron J. Seigo" <aseigo () olympusproject ! org>
Date:       2003-07-17 5:37:26
[Download RAW message or body]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wednesday 16 July 2003 08:30, arnaud.burlet@epfl.ch wrote:
> Hello,
>
> I implemented in KMail a way to take multiple email addresses from multiple
> messages and add them to the address book with only few mouse clicks...
>
> For an illustrated description please see
> http://www.realness.ch/~sts/kmail/

ah, a feature i've often missed! if i may be so bold as to offer some ideas 
and suggestions:

 o instead of "Take Addresses" in the context menu, how about "Add To Address 
Book" as it is in the reader window RMB?

 o instead of a button to click on to edit, why not provide in-line editting 
in the table cells of the name and email. further editting can be done via 
the address book itself, since that will likely be a rarer situation when 
adding from an email.

o provide some explanatory text in the "Take Addresses", er, "Add To Address 
Book" dialog to explain what's going on

 o how about using a listbox / listview instead of a table? a listview would 
be great since you could have multiple columns, inline editting AND 
checkboxes (QCheckListItem) ... it would also be more compact...

 o if a single email with just one address is selected, the multiple select 
box doesn't come up?

 o in the code, if the address already exists in the addressbook it pops up an 
error box saying it already exists. the dialog should inform the user of this 
situation and give them option to create a new record anyways (it may be a 
different person, same email; or may just want another record because they're 
an odd fellow ;), and a 'Skip This Address' button.

 o i haven't actually tried the code (JUST finished recompiling kdepim, don't 
feel like recompiling kmail right now ;), but when an address is inserted 
into the list, is the checkbox checked by default? if not, they probably 
should be.

 o seeing as the addressbook member isn't used until insertSelected is called 
(the user hits OK), should it's initialization be delayed until then to save 
on potentially unecessary loads (and reloads) of the addressbook?

 o a nice-to-have feature would be to optionally create disribution list from 
the addresses. perhaps as another context menu item: "Create Distribution 
List" which would allow one to providea name for the list via another widget 
in the listing dialog


- -- 
Aaron J. Seigo
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43

KDE: The 'K' is for 'kick ass'
http://www.kde.org       http://promo.kde.org/3.1/feature_guide.php
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.2-rc1-SuSE (GNU/Linux)

iD8DBQE/FjYX1rcusafx20MRAnkAAJwNn0lRcLbD1Wae9P5akC/FOa6VLwCffgMX
XVV7uoIyGehrTbvsmrscMAw=
=69DS
-----END PGP SIGNATURE-----
_______________________________________________
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]