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/#review77317
-----------------------------------------------------------


Moved to new repo: https://git.reviewboard.kde.org/r/122911/

- Gregor Mi


On March 1, 2015, 2:36 p.m., Gregor Mi wrote:
> 
> -----------------------------------------------------------
> 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.
> 
> 
> 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
> -----
> 
>   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/20150311/90eec692/attachment.htm>