Review Request 118719: Make Ctrl+I toggle the Filter Bar visibility

[Frank Reininghaus]

> On June 13, 2014, 7:29 a.m., Kai Uwe Broulik wrote:
> > Will it always close the filter bar when it is shown? I am a heavy user of this feature and I use Ctrl+I to focus it a lot. I think it should only hide again when it has focus, so you can still use it by keyboard alone.

Thanks Kai, that's a good point. That would be a reason not to make this change (hiding the bar while it is active is already possible with Esc).


- Frank


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


On June 13, 2014, 7:13 a.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/118719/
> -----------------------------------------------------------
> 
> (Updated June 13, 2014, 7:13 a.m.)
> 
> 
> Review request for Dolphin and KDE Usability.
> 
> 
> Repository: kde-baseapps
> 
> 
> Description
> -------
> 
> Users suggested in the forum thread http://forum.kde.org/viewtopic.php?f=285&t=121393 that the same shortcut should be used for showing and hiding the filter bar. This patch implements this behavior, and makes the "Show Filter Bar" action a "toggle" action to achieve that.
> 
> Ctrl+I now toggles the visibility of the filter bar, and a checkbox is added to the menu to show the current state of that action, see screenshots (the first one shows the current state, and the others the patched version with the filter bar shown/hidden). Additionally, Esc still closes the filter bar. I guess that we might want keep this behavior because some users might have got used to it, and Esc is sort of the standard key to get out of something.
> 
> A note on the implementation: I had to add the statement
> 
> emit showFilterBarChanged(true);
> 
> to the "true" branch in DolphinViewContainer::setFilterBarVisible(bool) to ensure that the state of the action is always consistent with the "visible" state of the filter bar. Without this change, it could become inconsistent in some situations. E.g., when enabling "Show Filter Bar" in the startup section of the settings dialog, the filter bar was shown, but the action was still unchecked.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/dolphinmainwindow.h cb97612 
>   dolphin/src/dolphinmainwindow.cpp 0ad224c 
>   dolphin/src/dolphinviewcontainer.cpp 57452b9 
> 
> Diff: https://git.reviewboard.kde.org/r/118719/diff/
> 
> 
> Testing
> -------
> 
> Ctrl+I hides the filter bar if it is shown.
> 
> 
> File Attachments
> ----------------
> 
> master-filter-bar-shown.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/06/13/467e2831-7e90-4561-9ea3-63238a7afc27__master-filter-bar-shown.png
> patched-filter-bar-shown.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/06/13/f45514f7-afd1-494e-96ec-bc6c24f0c88c__patched-filter-bar-shown.png
> patched-filter-bar-hidden.png
>   https://git.reviewboard.kde.org/media/uploaded/files/2014/06/13/9887897e-5d0a-44bf-8fa8-c04988c9cb80__patched-filter-bar-hidden.png
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

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