D26067: [KFilePlacesView] Add missing functionality required in order to be used by Dolphin again

Kai Uwe Broulik noreply at phabricator.kde.org
Tue Dec 17 15:34:18 GMT 2019


broulik created this revision.
broulik added reviewers: Frameworks, VDG, Dolphin.
Herald added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
broulik requested review of this revision.

REVISION SUMMARY
  This includes:
  
  - Rendering hidden items desaturated when in "show all" mode
  - A getter for "show all"
  - Signals for place activation/clicking
  - Signals for middle clicking a place (for e.g. open in a new tab)
  - Possibility to add custom actions to the context menu
  - Possibility to provide own storage teardown procedure
  - Changing view icon size is forwarded to delegates
  
  The delegates are also made somewhat more compact to match Dolphin and spacing inbetween sections increased.

TEST PLAN
  Try with branch `broulik/kfileplacesview` in Dolphin.
  
  - Can I remove a `Q_PRIVATE_SLOT`?
  
  - Currently when enabling "show all" and clicking a place, "show all" is deactivated again. Dolphin keeps this permanently. Should we change this?
  
  Dolphin (from left to right: current custom view, `KFilePlacesView` current look, `KFilePlacesView` more compact new look to match Dolphin)
  F7822829: Screenshot_20191217_162724.png <https://phabricator.kde.org/F7822829>
  Open dialog (from left to right: current look, more compact look, imho looks a bit crowded in conjunction with the file dialog :/0
  F7822826: Screenshot_20191217_162838.png <https://phabricator.kde.org/F7822826>

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D26067

AFFECTED FILES
  src/filewidgets/kfileplacesview.cpp
  src/filewidgets/kfileplacesview.h
  src/filewidgets/kfileplacesview_p.h

To: broulik, #frameworks, #vdg, #dolphin
Cc: kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20191217/78ec5b1a/attachment.htm>


More information about the kfm-devel mailing list