[Kde-pim] Review Request: Bugzilla status row in fancy headers

[Martin Gräßlin]

> On Jan. 8, 2013, 1:57 p.m., Laurent Montel wrote:
> > For me it will a good idea to add this patch. I didn't test it for the moment.
> > Will test it tomorrow.
> > We can't add it for 4.10
> > But perhaps that for 4.11 it will necessary to improve header settings. Perhaps add a dialogbox to define which extra header we want etc.
> > Will review it soon.
> > Regards.

yeah, of course for 4.11 and having some configuration sounds reasonable. I saw the code and all the checks and started to search in my KMail instance for the config dialog to enable the headers :-)


- Martin


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


On Jan. 8, 2013, 12:05 p.m., Martin Gräßlin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/108261/
> -----------------------------------------------------------
> 
> (Updated Jan. 8, 2013, 12:05 p.m.)
> 
> 
> Review request for KDEPIM.
> 
> 
> Description
> -------
> 
> If bugzilla headers are available the bugzilla product, component and status is shown in the fancy headers.
> 
> This is basically an idea I had to make my life easier in bug management. I often need to know whether a new comment is a closed bug.
> 
> As this is my first patch to kmail and I'm not really familiar with the source code please provide suggestion on how to get it into a mergable state. Though I don't mind if you say that it's unfit for inclusion - then I just carry around the patch.
> 
> 
> Diffs
> -----
> 
>   messageviewer/headerstrategy.cpp aecf717 
>   messageviewer/headerstyle.cpp 59506d1 
> 
> Diff: http://git.reviewboard.kde.org/r/108261/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> Email with the bugzilla status row
>   http://git.reviewboard.kde.org/r/108261/s/991/
> 
> 
> Thanks,
> 
> Martin Gräßlin
> 
>

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