Review Request 114480: Preferences button behaviour has been fixed and adjusted

David Edmundson david at davidedmundson.co.uk
Sun Dec 15 17:17:02 UTC 2013


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


Martin, can you have a look at this. Did you had a good reason for making it a QToolButton?

- David Edmundson


On Dec. 15, 2013, 5:10 p.m., Illya Kovalevskyy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/114480/
> -----------------------------------------------------------
> 
> (Updated Dec. 15, 2013, 5:10 p.m.)
> 
> 
> Review request for Telepathy and David Edmundson.
> 
> 
> Repository: ktp-contact-list
> 
> 
> Description
> -------
> 
> According to Google Code-In 2013 task (http://www.google-melange.com/gci/task/view/google/gci2013/5845533033758720) and a KDE Bug (https://bugs.kde.org/show_bug.cgi?id=316591):
> 
> Now, preferences tool icon is a QToolButton basically, so it's not an actual visual children (in terms of UI/UX) of the tool bar and its behaviour is broken: when you decrese contact list, so it hides it's not displayed inside additional action list (right arrow), because it's not an action.
> 
> This patch fixes it's behaviour by making this a QAction + adjusting some stuff
> 
> 
> Diffs
> -----
> 
>   main-widget.cpp 93574d9 
> 
> Diff: http://git.reviewboard.kde.org/r/114480/diff/
> 
> 
> Testing
> -------
> 
> Visually looks good and no compiler/runtime/console errors.
> 
> 
> File Attachments
> ----------------
> 
> Regular view
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/12/15/d6f2c653-37a5-461d-9624-4f6e5df67281__Window-Regular.png
> Window narrowed
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/12/15/21ef3aa3-a969-489c-938c-0af7cb5f3556__Window-Narrow.png
> Bugfix "in action"
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/12/15/e6348ed3-b722-49fe-95fe-9e8384b04214__Window-Exposed.png
> 
> 
> Thanks,
> 
> Illya Kovalevskyy
> 
>

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


More information about the KDE-Telepathy mailing list