Review Request 122352: dolphin: SpaceInfoToolsMenu: introduce KMoreTools

[Gregor Mi]

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

(Updated March 1, 2015, 2:36 p.m.)


Review request for Dolphin, Dominik Haumann, Emmanuel Pescosta, Eike Hein, and Michael Lutynski.


Changes
-------

- adapt to api changes
- make sure the configure dialog can be invoked without crash
- use new filelight desktop file (but does currently not work due to a potential issue in (or wrong usage of) KService, see https://bugs.kde.org/show_bug.cgi?id=344614)


Repository: kde-baseapps


Description
-------

As said in https://git.reviewboard.kde.org/r/121478/ I created a set of classes called "KMoreTools" that should help with "more tools" scenarios like this one. The corresponding wiki page is now https://community.kde.org/Scratchpad/KMoreToolsFramework. The code examples there are not up-to-date, the latest demo code is in this RR.

Though the classes around KMoreTools are designed so that it could be used in any KDE application, for this RR I added it to dolphin because SpaceInfoToolsMenu is a first use-case and proof-of-concept for KMoreTools.

>From the standpoint of SpaceInfoToolsMenu (dolphin/src/statusbar/spaceinfotoolsmenu.cpp) the following has improved:

- The code for creating the menu is shorter and better readable.
- Not-installed applications are now automatically placed to the new "More" submenu under the "Not installed" menu section.
- The "More" menu is not created when not needed (e.g. when all applications in the menu are installed)
- A new application was added: gparted, which is placed in the "More" submenu by default.
- For "Not installed" applications there is a menu item that opens the applications homepage in the default browser or a hint if no such information was provided.
- Slightly reduced amount of needed translations since the not-installed handling is done by KMoreTools

There are some discussion points like the installation location of the locally-copied desktop files (kmt-desktopfiles).

I propose to use KMoreTools in its current state (after review) as replacement for the current SpaceInfoTools implementation. If it is considered useful it could be somewhen moved to a common module


Diffs (updated)
-----

  dolphin/src/CMakeLists.txt 94aa5a51caa7bad8eb2b3427a5b6518124e097ae 
  dolphin/src/statusbar/kmt-desktopfiles/filelight.desktop PRE-CREATION 
  dolphin/src/statusbar/kmt-desktopfiles/gparted.desktop PRE-CREATION 
  dolphin/src/statusbar/kmt-desktopfiles/gparted.png PRE-CREATION 
  dolphin/src/statusbar/kmt-desktopfiles/kdf.desktop PRE-CREATION 
  dolphin/src/statusbar/kmt-desktopfiles/org.kde.filelight.desktop PRE-CREATION 
  dolphin/src/statusbar/spaceinfotoolsmenu.h 3ca2e184f03b31d10d661c5a931644485ffc46e6 
  dolphin/src/statusbar/spaceinfotoolsmenu.cpp bce5ba11d439ee3950062abc779120e75a864326 
  dolphin/src/statusbar/statusbarspaceinfo.h 326c419ab3dc709f5bc4ab23f73e727d1106b3a4 
  dolphin/src/statusbar/statusbarspaceinfo.cpp 127641e601d1468479ef925f9539ecd31dc12fb7 

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


Testing
-------


Thanks,

Gregor Mi

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