Patch for 224859

Raphael Kubo da Costa kubito at gmail.com
Fri Apr 9 22:50:27 CEST 2010


On Wednesday 07 April 2010 17:04:37 Raphael Kubo da Costa wrote:
> On Wednesday 07 April 2010 06:39:50 Eike Lange wrote:
> > Hi Raphael
> > 
> > Raphael Kubo da Costa schrieb:
> > > I'd rather use KAction instead of QAction, and add the related
> > > #includes. Other than that, the patch looks fine.
> > 
> > Done and attached.
> > 
> > > Do you have an svn account?
> > 
> > No,not at the moment. Maybe, I ask for one, if I have some more
> > experiences.
> > 
> > Eike
> 
> Hey Eike,
> 
> I was thinking about this again, and I think it's better to use
> KStandardAction::quit instead of creating a KAction and assign 
Ctrl+Q to it
> -- if the user changes the standard shortcut to quit applications, 
KTimer
> should follow accordingly.
> 
> This way, I suggest using KAction *exit = KStandardAction::quit(this,
> SLOT(exit()), this); for example.
> 
> Cheers,
> Raphael

Hello?


More information about the Kde-utils-devel mailing list