[KDE Usability] Review Request 118719: Make Ctrl+I toggle the Filter Bar visibility

[Thomas Pfeiffer]

> 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.
> 
> Frank Reininghaus wrote:
>     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).

Heiko and me talked about search again a few days ago, and this issue came up as well. We looked at existing applications (both KDE and non-KDE) and found that focusing the search field when the shortcut for opening it is pressed a second time is the de-facto standard. We found it to be useful and therefore would not want to deviate from it.

Therefore I agree with Kai that we should not make this change. Sorry for your work on this going to waste, but when one thinks it through, being able to focus the field is more important than being able to hide it (especially since hiding with escape is still possible).


- Thomas


-----------------------------------------------------------
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/34677762/attachment.htm>
-------------- next part --------------
_______________________________________________
kde-usability mailing list
kde-usability at kde.org
https://mail.kde.org/mailman/listinfo/kde-usability