Review Request: Fix for bug 300248

[Todd Jennings]

> On Sept. 5, 2012, 8:59 a.m., Frank Reininghaus wrote:
> > Thanks for the patch!
> > 
> > About the button text: I'm not sure either what the optimal solution is. Maybe "More Options" and "Hide Options"?
> > 
> > Another thing: I was wondering if we should put both the icon and the new text on the button. Maybe this would make it more obvious that this button is different from the others? Just to make sure that noone overlooks the button at first sight.
> > 
> > @Kai: I haven't worked on any of the search-related code myself, but it seems that there is indeed not way to search for tags in the new interface. See also https://bugs.kde.org/show_bug.cgi?id=304406

Is the "+" icon the correct icon?  This isn't really adding something, it is expanding and contracting a hidden section.  As far as I can recall, other KDE applications use arrow-down-double and arrow-up-double for this sort of behavior (see Kate for example).

Also, should it have a tooltip?


- Todd


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


On Sept. 4, 2012, 1:27 p.m., Panos Kanavos wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/106325/
> -----------------------------------------------------------
> 
> (Updated Sept. 4, 2012, 1:27 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Description
> -------
> 
> Provides a possible improvement to Dolphin's additional search criteria button, according to the request in https://bugs.kde.org/show_bug.cgi?id=300248.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/search/dolphinsearchbox.h ee9987a 
>   dolphin/src/search/dolphinsearchbox.cpp 28f1f1a 
> 
> Diff: http://git.reviewboard.kde.org/r/106325/diff/
> 
> 
> Testing
> -------
> 
> The modified button behaves at it should.
> 
> 
> Thanks,
> 
> Panos Kanavos
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20120905/685a019a/attachment.htm>