Review Request 119411: Plasmate-Kdev: Port plasmate to KDevPlatform's shell part 1
Aleix Pol Gonzalez
aleixpol at kde.org
Thu Aug 7 16:37:09 UTC 2014
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119411/#review64004
-----------------------------------------------------------
File Attachment: How plasmate should look like in the end - plasmate_ui.png
<https://git.reviewboard.kde.org//r/119411/#fcomment234>
Didn't you have an editor for metadata files?
File Attachment: How plasmate should look like in the end - plasmate_ui.png
<https://git.reviewboard.kde.org//r/119411/#fcomment235>
Maybe you don't want the console?
File Attachment: How plasmate should look like in the end - plasmate_ui.png
<https://git.reviewboard.kde.org//r/119411/#fcomment236>
Have you integrated plasma documentation there?
File Attachment: How plasmate should look like in the end - plasmate_ui.png
<https://git.reviewboard.kde.org//r/119411/#fcomment237>
Wouldn't it make sense to integrate Package into Project?
What are the differences?
File Attachment: How plasmate should look like in the end - plasmate_ui.png
<https://git.reviewboard.kde.org//r/119411/#fcomment238>
No Text?
File Attachment: How plasmate should look like in the end - plasmate_ui.png
<https://git.reviewboard.kde.org//r/119411/#fcomment239>
2 Run menus?
In fact, plasmate probably wants to define its own menus, right?
plasmate/app/main.cpp
<https://git.reviewboard.kde.org/r/119411/#comment44669>
Patches are welcome.
- Aleix Pol Gonzalez
On Aug. 2, 2014, 10:20 a.m., Giorgos Tsiapaliokas wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119411/
> -----------------------------------------------------------
>
> (Updated Aug. 2, 2014, 10:20 a.m.)
>
>
> Review request for Plasma.
>
>
> Repository: plasmate
>
>
> Description
> -------
>
> I have separated this task to two reviews, because there are also some
> `git mv` in order to have a decent directory structure which I don't have
> included in this review for readability.
>
> Below is the list of commits from which this review has been made.
>
> ----
>
> * use some private members and improve readability
>
> ----
> * Every project must have a .plasmate file in order to pick the Manager.
>
> ----
> * Initial commit for porting plasmate to kdevplatform
>
> We introduce a new class named PlasmateApp which will
> show the Startpage and hide it, when it must do it.
>
>
> Diffs
> -----
>
> plasmate/CMakeLists.txt 1a6ce87
> plasmate/app/main.cpp PRE-CREATION
> plasmate/app/plasmateapp.h PRE-CREATION
> plasmate/app/plasmateapp.cpp PRE-CREATION
> plasmate/app/plasmateextention.h PRE-CREATION
> plasmate/app/plasmateextention.cpp PRE-CREATION
> plasmate/app/plasmateui.rc PRE-CREATION
> plasmate/main.cpp 633c4cc
> plasmate/plasmateui.rc 41f6025
> plasmate/startpage.h 4c77e29
> plasmate/startpage.cpp a65a2cc
>
> Diff: https://git.reviewboard.kde.org/r/119411/diff/
>
>
> Testing
> -------
>
> You can also test this review from
>
> url: http://quickgit.kde.org/?p=clones%2Fplasmate%2Ftsiapaliwkas%2Fplasmate-kdevplatform.git
> git: git://anongit.kde.org/clones/plasmate/tsiapaliwkas/plasmate-kdevplatform.git
> branch: r/119411
>
>
> File Attachments
> ----------------
>
> How plasmate should look like in the end
> https://git.reviewboard.kde.org/media/uploaded/files/2014/08/02/f0937ff2-53b9-4adc-9b5c-5a758a32ccd5__plasmate_ui.png
>
>
> Thanks,
>
> Giorgos Tsiapaliokas
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140807/63080cc2/attachment-0001.html>
More information about the Plasma-devel
mailing list