-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116846/#review53161
-----------------------------------------------------------


Good stuff! Can you try also setting a text to that action so that when you 
expand the toolbar, it actually says "Settings" next to the icon? Or any better 
string? Then upload again the screenshot with the toolbar extended.

/me is also wondering if we can make it proper submenu, without the need to 
click on the button to expand it...

- Martin Klapetek


On March 17, 2014, 6:45 a.m., Nilesh Suthar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116846/
> -----------------------------------------------------------
> 
> (Updated March 17, 2014, 6:45 a.m.)
> 
> 
> Review request for Telepathy and Martin Klapetek.
> 
> 
> Bugs: 316591
>     http://bugs.kde.org/show_bug.cgi?id=316591
> 
> 
> Repository: ktp-contact-list
> 
> 
> Description
> -------
> 
> Implemented SettingWidgetAction class by extending QWidgetAction. As during 
> narrowing QToolBar populate only actions.So now by Extending QWidgetAction.We 
> have implement Action with custom wigdet.
> 
> 
> Diffs
> -----
> 
>   main-widget.h aa69daa 
>   main-widget.cpp 27a941d 
> 
> Diff: https://git.reviewboard.kde.org/r/116846/diff/
> 
> 
> Testing
> -------
> 
> Working fine.
> 
> 
> File Attachments
> ----------------
> 
> Regular View.png
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2014/03/17/b249cfb9-7a6f-47ac-bb5c-f50eb23872ed__Regular_View.png
> Window narrowed.png
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2014/03/17/ecc5af4a-0766-4608-8596-fe39da54a950__Window_narrowed.png
> Window narrow arrow clicked.png
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2014/03/17/c77d661f-0480-4f22-b677-417e215f7f01__Window_narrow_arrow_clicked.png
> Window narrow setting clicked.png
>   
> https://git.reviewboard.kde.org/media/uploaded/files/2014/03/17/a3ec661f-52e2-48a4-8073-c276d0093e60__Window_narrow_setting_clicked.png
> 
> 
> Thanks,
> 
> Nilesh Suthar
> 
>

_______________________________________________
KDE-Telepathy mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/kde-telepathy

Reply via email to