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

[Francesco Nwokeka]

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



main-widget.h
<http://git.reviewboard.kde.org/r/101121/#comment2338>

    variable naming.
    
    actionAddContact -> addContactAction
    
    etc


- Francesco


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/70fb226f/attachment.htm