[Kde-pim] Review Request: Apply template when switching identity in message composer

[Allen Winter]

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


I don't see any obvious problems.  Looks very good.

- Allen


On 2009-06-13 12:31:24, Jonathan Armond wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/833/
> -----------------------------------------------------------
> 
> (Updated 2009-06-13 12:31:24)
> 
> 
> Review request for KDE PIM.
> 
> 
> Summary
> -------
> 
> Apply new template when switching identity in the message composer, as long the message has not been modified. NB: changing identity no longer marks message as modified.
> 
> Apply template to Forward As Attachment messages.
> 
> First attempt at fixing bug 143199.
> 
> 
> This addresses bugs 143199, 179754 and 186843.
>     https://bugs.kde.org/show_bug.cgi?id=143199
>     https://bugs.kde.org/show_bug.cgi?id=179754
>     https://bugs.kde.org/show_bug.cgi?id=186843
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdepim/kmail/composer.h 980307 
>   /trunk/KDE/kdepim/kmail/kmcommands.cpp 980307 
>   /trunk/KDE/kdepim/kmail/kmcomposewin.h 980307 
>   /trunk/KDE/kdepim/kmail/kmcomposewin.cpp 980307 
>   /trunk/KDE/kdepim/kmail/kmkernel.cpp 980307 
>   /trunk/KDE/kdepim/kmail/kmmainwidget.cpp 980307 
>   /trunk/KDE/kdepim/kmail/templateparser.h 980307 
>   /trunk/KDE/kdepim/kmail/templateparser.cpp 980307 
> 
> Diff: http://reviewboard.kde.org/r/833/diff
> 
> 
> Testing
> -------
> 
> Template properly parsed for new, reply, reply all and forward. Switches between custom, global and no templates properly. Doesn't switch after modifying message.
> 
> 
> Thanks,
> 
> Jonathan
> 
>

_______________________________________________
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/