Review Request: Add "Remove user from this group" and use Konvi's url detector in the context menu links menu

[David Edmundson]

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



main-widget.cpp
<http://git.reviewboard.kde.org/r/103118/#comment6976>

    QAction -> KAction
    
    Fix your Capitalisation. Either have It all Title case or Not.
    (Actually I think UI rules state it should be sentence case.)



main-widget.cpp
<http://git.reviewboard.kde.org/r/103118/#comment6978>

    Telepathy does support blocking. In fact we use it in 'remove'. (there was an issue in butterfly at one point, but that's fairly irrelevant now.)
    
    Can we sort this out in a separate commit sometime.
    
    



main-widget.cpp
<http://git.reviewboard.kde.org/r/103118/#comment6977>

    I know this is technically just moved code, but QAction->KAction
    


- David Edmundson


On Nov. 12, 2011, 12:31 p.m., Martin Klapetek wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103118/
> -----------------------------------------------------------
> 
> (Updated Nov. 12, 2011, 12:31 p.m.)
> 
> 
> Review request for Telepathy and Dario Freddi.
> 
> 
> Description
> -------
> 
> As description
> 
> 
> Diffs
> -----
> 
>   main-widget.h f602a3a 
>   main-widget.cpp ab37eb0 
> 
> Diff: http://git.reviewboard.kde.org/r/103118/diff/diff
> 
> 
> Testing
> -------
> 
> Works.
> 
> 
> Thanks,
> 
> Martin Klapetek
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20111112/51a92289/attachment.html>