Review Request 127054: Add a Filter Toolbar Button for Dolphin

[arnav dhamija]

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

(Updated March 16, 2016, 4:53 p.m.)


Review request for Dolphin, KDE Usability and Emmanuel Pescosta.


Changes
-------

Fixed toggle behavior by implmenting a QShortcut.


Repository: dolphin


Description
-------

This patch was created to implement the feature suggested here: [https://todo.kde.org/?controller=task&action=show&task_id=966](https://todo.kde.org/?controller=task&action=show&task_id=966)

This patch adds a Filter Button to the toolbar of Dolphin. This button can be toggled to show/hide the Filter Bar.

In this patch, one might notice that there is now show_filter_bar and show_filter_bar_button in dolphinui.rc file. This was done because I found that using the same action for the menu entry and the toolbar entry would cause something to break (eg, shortcuts would stop working). 

A screenshot of the toolbar is also attached.


Diffs (updated)
-----

  dolphinmainwindow.h 7003e94 
  dolphinmainwindow.cpp f7a7613 
  dolphinviewcontainer.h 62f9110 
  dolphinviewcontainer.cpp 8fea3ba 

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


Testing
-------

manual


File Attachments (updated)
----------------

dolphintoolbar.png
  https://git.reviewboard.kde.org/media/uploaded/files/2016/02/12/d936e5d0-557c-45c5-9a2d-f849e5d284a6__dolphintoolbar.png
patch.txt
  https://git.reviewboard.kde.org/media/uploaded/files/2016/03/16/1eb43168-65dd-410b-a74b-f895f3035c83__patch.txt


Thanks,

arnav dhamija

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