[Kde-pim] Review Request: new kmail filter : add to address book

Bruno Bigras bigras.bruno at gmail.com
Mon Jul 27 11:10:23 BST 2009


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/1038/
-----------------------------------------------------------

(Updated 2009-07-27 10:10:23.725029)


Review request for KDE PIM.


Summary
-------

This is a port of a commit by Christian Schaarschmidt https://bugs.kde.org/show_bug.cgi?id=47333#c19 from the kdepim-3.5.5+ branch.

It add an "add to address book" filter action.

I tried to make the ui look better when the window is not maximized (see the two screenshots) but the first combo box (the one to select "From", "To", "CC", "BCC") don't look so good in the center.


This addresses bug 47333.
    https://bugs.kde.org/show_bug.cgi?id=47333


Diffs
-----

  /trunk/KDE/kdepim/kmail/kmfilteraction.cpp 1002846 
  /trunk/KDE/kdepim/kmail/kmfilterdlg.h 1002846 
  /trunk/KDE/kdepim/kmail/kmfilterdlg.cpp 1002846 

Diff: http://reviewboard.kde.org/r/1038/diff


Testing
-------

I tested this successfully when receiving emails and when sending to multiples recipients.


Screenshots
-----------

before
  http://reviewboard.kde.org/r/1038/s/143/
after
  http://reviewboard.kde.org/r/1038/s/144/
after-QGridLayout
  http://reviewboard.kde.org/r/1038/s/148/
Proposed solution
  http://reviewboard.kde.org/r/1038/s/154/


Thanks,

Bruno

_______________________________________________
KDE PIM mailing list kde-pim at kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/



More information about the kde-pim mailing list