Review Request 121478: dolphin: add menu to SpaceInfo widget for detailled disk usage information

[Gregor Mi]

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

(Updated Jan. 10, 2015, 11:47 a.m.)


Review request for Dolphin and Emmanuel Pescosta.


Changes
-------

Fixed issues: remove the empty file, fix include statment order, restructure "if" statements, add more of "const auto"


Repository: kde-baseapps


Description
-------

Adds a menu when left-clicked on the SpaceInfo widget in the status bar with the following entries:

(1) KDiskFree
(2) Filelight
(3) Filelight device
(4) Filelight location

see screenshot: http://wstaw.org/m/2014/12/13/plasma-desktopqO2157.png

Details on the menu entries:

(1) starts KDiskFree (useful tool, hard to find when unknown by new user)
(2) starts Filelight without parameters
(3) starts Filelight with current mount point
(4) starts Filelight with current location

If KDiskFree or Filelight is not installed the corresponding menu action is disabled with a hint "[Not installed]".


Diffs (updated)
-----

  dolphin/src/CMakeLists.txt f132e2844d54d08fa99b6c5ef8c5da6dfdb07514 
  dolphin/src/statusbar/spaceinfotoolsmenu.h PRE-CREATION 
  dolphin/src/statusbar/spaceinfotoolsmenu.cpp PRE-CREATION 
  dolphin/src/statusbar/statusbarspaceinfo.h 5d571beba9b936c52dd3279e370c39a1bfd50827 
  dolphin/src/statusbar/statusbarspaceinfo.cpp 046cbbb23072648aece5677defb1b9074baa9482 

Diff: https://git.reviewboard.kde.org/r/121478/diff/


Testing
-------

Clicked all the menu entries and they work as described.


File Attachments
----------------

2015-01-09-dolphin-diskspace.png
  https://git.reviewboard.kde.org/media/uploaded/files/2015/01/09/b87a29dc-25bc-4ce6-a5be-9fcd405fe455__2015-01-09-dolphin-diskspace.png


Thanks,

Gregor Mi

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