Review Request 129533: Dolphin: expand all one level, expand all, and collapse all in details view mode

[Heiko Tietze]

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



Your \ is accessed on a German keyboard layout per AltGr (right alt key), would be a really unusual combination with ctrl. The idea was to have the division (ctrl+/) and the multiplication, and to alterate this 'normal' ctrl+ * so users have to remember just the * , which is secured further for the hazardous expand-all-function. Both are easily accesible on the numblock unlike special keys on the various layouts.
(Guess these keys are configurable in the shortcuts dialog and we discuss the default only.)

The context menu is the first place to look for functions. Keep your stuff there, it doesn't hurt. And also keep in mind that users can hide the menu- and the toolbar.

The toolbar buttons need icons, ask Andreas Kainz for support. And in my opinion it's way too much text for a convenience feature. It draws a lot of attention to these functions just because of the verbose labels. Solution 1: icon only, Solution 2: shorter labels (Expand One, Expand Full, Collapse [Full]), Solution 3: Use a split/menu button and list the functions there, Solution 4: Ignore my objection ;-).

Looptest: "Could not enter" ist not really actionable. How about 'Trees can have up to 256 levels. Dolphin cannot handle this depth, use file system functions' (this is just a draft to illustrate the idea of actionable).

PS: If you worry about the shortcuts we could run a quick poll.

- Heiko Tietze


On Nov. 27, 2016, 7:12 a.m., Don Nguyen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/129533/
> -----------------------------------------------------------
> 
> (Updated Nov. 27, 2016, 7:12 a.m.)
> 
> 
> Review request for Dolphin and KDE Usability.
> 
> 
> Bugs: 196772
>     https://bugs.kde.org/show_bug.cgi?id=196772
> 
> 
> Repository: dolphin
> 
> 
> Description
> -------
> 
> Implemented functions:  Expand All One Level, Expand All, and Collapse All.  They are available via the context menu in details view mode as well as custom toolbar buttons as Thomas explained.  They are also available under the View and Control menus and are disabled when not in details view mode.  See screenshots.  I think they should be removed from the context menus - they are in the context menus now for usability testing.
> 
> I did not use the Ctrl+Shift+* shortcut because on my laptop there is no numpad so it's not possible to generate that shortcut.
> 
> 
> Diffs
> -----
> 
>   src/dolphincontextmenu.h f67300d 
>   src/dolphincontextmenu.cpp ed3f643 
>   src/dolphinmainwindow.cpp e5103fd 
>   src/dolphinui.rc f197af4 
>   src/kitemviews/kfileitemmodel.h 8a0df72 
>   src/kitemviews/kfileitemmodel.cpp 05f85a6 
>   src/views/dolphinview.h 0b0d819 
>   src/views/dolphinview.cpp 4105628 
>   src/views/dolphinviewactionhandler.h eb375b5 
>   src/views/dolphinviewactionhandler.cpp 858f929 
> 
> Diff: https://git.reviewboard.kde.org/r/129533/diff/
> 
> 
> Testing
> -------
> 
> make test passes
> manual testing including:
> expand all on an Android SDK folder which yielded in 3180 folders, and 30993 files
> collapse all on above situation
> expand all on a directory that contains a symbolic link to a directory containing itself - operation will stop with error when expansion reaches a path that KCoreDirLister doesn't accept(see looptest screenshot) 
> changing directories in the middle of an expand all operation
> changing directories in the middle of a restore expanded directories operation 
> canceling in the middle of an expand all operation
> canceling in the middle of a restore expanded directories operation
> canceling in the middle of an expand all operation or restore expand directory operation, and then expanding an item manually with mouse click
> collapse all in the middle of an expand all operation
> collapse all in the middle of a restore expanded directories operation
> switched view modes and verified items are enabled only in details view mode
> 
> 
> File Attachments
> ----------------
> 
> mainwindow
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/25/b4a961b6-3ed1-4115-876d-5d804c344f19__mainwindow.png
> contextmenu
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/25/2b6a89a8-722e-4f88-a1ca-5da3f6c89140__contextmenu.png
> looptest
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/25/f02b7e85-d0b3-4153-ae0d-e86a4ddfc916__looptest.png
> enabled
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/27/fbd70131-1f4c-49bc-8f8c-cdba8f67aa07__mainwindowenabled.png
> disabled
>   https://git.reviewboard.kde.org/media/uploaded/files/2016/11/27/3cfbe3d9-7a45-436d-b3a0-46d65393ce62__mainwindowdisabled.png
> 
> 
> Thanks,
> 
> Don Nguyen
> 
>

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