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

List:       kde-pim
Subject:    Re: [Kde-pim] Review Request: Bugzilla status row in fancy headers
From:       Martin_Gräßlin <kde () martin-graesslin ! com>
Date:       2013-01-08 14:43:06
Message-ID: 20130108144306.27924.41900 () vidsolbach ! de
[Download RAW message or body]



> 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@kde.org
https://mail.kde.org/mailman/listinfo/kde-pim
KDE PIM home page at http://pim.kde.org/


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