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

[Don Nguyen]

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

(Updated Jan. 23, 2017, 10:26 p.m.)


Review request for Dolphin and KDE Usability.


Changes
-------

this patch moves the expand/collapse functions into a submenu


Bugs: 196772
    https://bugs.kde.org/show_bug.cgi?id=196772


Repository: dolphin


Description
-------

Implemented functions:  Expand, Expand Full, and Collapse All.  They are available under the View and Control menus as well as custom toolbar buttons as Thomas explained and are disabled when not in details view mode.  They are also available via the context menu only in details view mode.  See screenshots.  
Expand - If there is a selection, expand the selection once.  Otherwise expand everything once.  
Expand Full - If there is a selection, expand the selected items completely.  Otherwise expand everything completely.
Collapse All - Collapse everything.


Diffs (updated)
-----

  src/dolphincontextmenu.h f67300d 
  src/dolphincontextmenu.cpp bedb4cb 
  src/dolphinmainwindow.cpp 1dcae7e 
  src/dolphinui.rc f197af4 
  src/kitemviews/kfileitemmodel.h dc5746a 
  src/kitemviews/kfileitemmodel.cpp 7e14baf 
  src/tests/kfileitemmodeltest.cpp 59ff630 
  src/views/dolphinview.h fbe3a63 
  src/views/dolphinview.cpp f5ba63d 
  src/views/dolphinviewactionhandler.h eb375b5 
  src/views/dolphinviewactionhandler.cpp 273bcdd 

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 (updated)
----------------

looptest
  https://git.reviewboard.kde.org/media/uploaded/files/2016/11/25/f02b7e85-d0b3-4153-ae0d-e86a4ddfc916__looptest.png
context-menu-disabled
  https://git.reviewboard.kde.org/media/uploaded/files/2016/12/04/1fcc4412-e5f2-4c31-92bf-b828eea938f7__contextdisabled.png
context-menu-with-selection
  https://git.reviewboard.kde.org/media/uploaded/files/2016/12/04/8a4709e5-d674-4f74-8cbc-a71841db7c36__selectioncontextenabled.png
view-menu-enabled
  https://git.reviewboard.kde.org/media/uploaded/files/2016/12/04/1ddaa88d-e0aa-441a-a840-5e2cdc06fbf2__viewmenuenabled.png
view submenu
  https://git.reviewboard.kde.org/media/uploaded/files/2017/01/23/af06ef13-a5b5-4b09-99dc-a0972fdde63c__viewsubmenu.png
context submenu
  https://git.reviewboard.kde.org/media/uploaded/files/2017/01/23/9abdf55d-7846-4725-801c-cbe17e353dd3__contextsubmenu.png


Thanks,

Don Nguyen

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