Review Request: Use KAction instead of QAction in telepathy-contact-list

[Nicolas Lécureuil]

> On April 14, 2011, 1:46 p.m., Francesco Nwokeka wrote:
> > main-widget.h, lines 106-109
> > <http://git.reviewboard.kde.org/r/101121/diff/1/?file=14381#file14381line106>
> >
> >     variable naming.
> >     
> >     actionAddContact -> addContactAction
> >     
> >     etc

fixed and commited this way


- Nicolas


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


On April 14, 2011, 1:28 p.m., Nicolas Lécureuil wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/101121/
> -----------------------------------------------------------
> 
> (Updated April 14, 2011, 1:28 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Summary
> -------
> 
> This patch remove the QActions from the ui file and add them on the cpp file to allow us to use KAction and KStandardShortcut and thus use the standard KDE shortcuts.
> 
> 
> Diffs
> -----
> 
>   main-widget.h 18f97ee 
>   main-widget.cpp 7fe200c 
>   main-widget.ui d000b9b 
> 
> Diff: http://git.reviewboard.kde.org/r/101121/diff
> 
> 
> Testing
> -------
> 
> I tested with standard shortcut ( Ctrl +F ) + changed the shortcut in systemsettings and retested in telepathy-contact-list to see that it was using new shortcut ( Ctrl + J ) for my test.
> 
> 
> Thanks,
> 
> Nicolas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/kde-telepathy/attachments/20110414/52035e17/attachment.htm