[Kde-pim] [patch] kdepim/kdgantt1 - Appointment not in visible range

[David Faure]

On Wednesday 16 July 2008, Jaroslaw Staniek wrote:
> David Faure said the following, On 2008-07-16 13:03:
> > On Wednesday 16 July 2008, Jaroslaw Staniek wrote:
> >> for review
> >>
> >> Description:
> >> http://intevation.de/roundup/kolab/issue2881
> >>
> >> The fix is basically about porting virtual show() to Qt4's showEvent().
> >> KDGanttView::show() was never called, and thus neither was centerTimeline(), 
> >> so the view was not centered on the appointment initially.
> >>
> >> I did the same change for close()/closeEvent() BTW.
> > 
> > When porting from show to showEvent, one should add if (!event->isSpontaneous()) { ... }
> > 
> 
> Thanks for the note, attached updated patch.

Looks good to me now.

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).
_______________________________________________
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/