Review Request: Load appwizard templates from files

Milian Wolff mail at milianw.de
Wed May 23 19:58:35 UTC 2012


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/105009/#review14097
-----------------------------------------------------------


patchwise and ui-wise I think this is fine - whats missing is of course a way to upload file templates from within kdevelop (or at least a sane way to create new templates) - without that, I think this addition is not so useful yet, is it?


plugins/appwizard/projecttemplatesmodel.cpp
<http://git.reviewboard.kde.org/r/105009/#comment11148>

    imo this debug output is not really useful and should be removed before landing this patch eventually


- Milian Wolff


On May 23, 2012, 11:46 a.m., Miha Čančula wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105009/
> -----------------------------------------------------------
> 
> (Updated May 23, 2012, 11:46 a.m.)
> 
> 
> Review request for KDevelop, Milian Wolff and Alexander Dymo.
> 
> 
> Description
> -------
> 
> Implement loading templates from local file. Lodaded templates are immediately selected for convenience. Their descriptions are extracted, but their icons are not. I will add icon-extraction after the initial code is reviewed. The archive files are copied to a ~/.kde/share/apps/kdevappwizard/templates/ where the existing code picks them up. 
> 
> Since I was already adding buttons, I added one for getting new template with KNS as well. It shows a download dialog and refreshes the views, but since no category is defined yet on kde-files.org, it shows all unrelated items. 
> 
> Note that this code is currently in the "project-templates" branch. I realize it's not ready to be merged just yet (due to missing functionality), but I wanted comments regarding the implementation and user interface. 
> 
> PS: The source files in appwizard files list a bunch of contributors, who should I add as reviewers?
> 
> 
> Diffs
> -----
> 
>   plugins/appwizard/CMakeLists.txt 0e2119197765ad514b8827fda5afe7254f8ea788 
>   plugins/appwizard/kdevappwizard.knsrc PRE-CREATION 
>   plugins/appwizard/projectselectionpage.h 745a1efa1d00c2ac4fe215b7242cf2ed09981e9e 
>   plugins/appwizard/projectselectionpage.cpp 373c66f7e1218858651d38678cc1010b4764308a 
>   plugins/appwizard/projectselectionpage.ui 7a33fdd2c68831776ea5dc8d7259afa831854a41 
>   plugins/appwizard/projecttemplatesmodel.h cbd91e976e1472dc29d9bc263268bf71d86ec229 
>   plugins/appwizard/projecttemplatesmodel.cpp 1fe09efec92d85c71679648408e8972b1ddd4971 
> 
> Diff: http://git.reviewboard.kde.org/r/105009/diff/
> 
> 
> Testing
> -------
> 
> Tested locally, archives shipped with KAppTemplate are loaded successfully. 
> 
> 
> Screenshots
> -----------
> 
> New buttons in template selection
>   http://git.reviewboard.kde.org/r/105009/s/575/
> 
> 
> Thanks,
> 
> Miha Čančula
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20120523/1cf8cbaa/attachment.html>


More information about the KDevelop-devel mailing list