Review Request 123883: DolphinSearchBox: Add a "More search tools..." menu button

Gregor Mi codestruct at posteo.org
Fri Jan 22 11:29:18 GMT 2016


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

(Updated Jan. 22, 2016, 11:29 a.m.)


Review request for Dolphin and Emmanuel Pescosta.


Changes
-------

sorry again for the late response. Since the upgrade of reviewboard your code comments seem to have lost the correct context. I start with sending the latest diff and then I try to figure out the mentioned issues.


Repository: dolphin


Description
-------

Adds a "More search tools..." button to the DolphinSearchBox.

Additionally, moves the "More/Fewer options" button from right to left to reduce the mouse travelling distance when the dolphin is maximized on a large screen (see screenshots)

The current location url will be passed to the search tool to set the initial search root directory.


Diffs (updated)
-----

  src/search/dolphinsearchbox.cpp 743d9113a9ae0f7820d1bed594462a208b2883c3 

Diff: https://git.reviewboard.kde.org/r/123883/diff/


Testing
-------


File Attachments
----------------

Currently there is a long mouse traveling distance
  https://git.reviewboard.kde.org/media/uploaded/files/2015/05/23/8d717a70-215d-4587-89d0-2fee89263731__MoreOptionsTravellingDistance.png
"More search tools" and the "More/fewer options" buttons moved more to the left
  https://git.reviewboard.kde.org/media/uploaded/files/2015/05/23/3009c4e7-5666-4377-bf39-5ec1967858e6__MoreSearchTools.png


Thanks,

Gregor Mi

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


More information about the kfm-devel mailing list