Review Request: Plasmate: Add Tabbox support to the startpage

Martin Gräßlin kde at martin-graesslin.com
Thu May 24 06:46:27 UTC 2012


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



startpage.cpp
<http://git.reviewboard.kde.org/r/105011/#comment11158>

    I would use a kwin icon for TabBox and I think TabBox is a really, really bad name.
    
    I tend to call it "Window Switcher", the KCM is called "Task Switcher"



startpage.cpp
<http://git.reviewboard.kde.org/r/105011/#comment11159>

    be aware of correct writing: KWin/TabBox would be the way it is called inside KWin.



templates/mainTabbox.qml
<http://git.reviewboard.kde.org/r/105011/#comment11160>

    you should fill in the copyright :-)


- Martin Gräßlin


On May 23, 2012, 5:20 a.m., Antonis Tsiapaliokas wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105011/
> -----------------------------------------------------------
> 
> (Updated May 23, 2012, 5:20 a.m.)
> 
> 
> Review request for kwin, Plasma and Martin Gräßlin.
> 
> 
> Description
> -------
> 
> Hello,
> 
> This is the first task of my GSoC. I have add the tabbox support to the startpage of the plasmate.
> 
> NOTES:
> 
> 1)The service type of the tabbox is "Kwin/Tabbox,Plasma/Applet", because the Plasma::PackageStructure requires the "Plasma/Applet" in order to be able to be used. Also the plasmapkg and the plasmoidview require the "Plasma/Applet" service type.
> 2)Some lines doesn't have any differences because i have remove some whitespaces and "tabs"... 
> 
> ISSUES:
> 
> 1)The icons for the tabbox are wrong. I have some issues with my PCs and i cannot open a new session of the KDE.
> So i wasn't able to find the icon. Sorry for that.
> 2)The template of the tabbox that i have put is located in the kde-workspace/kwin/kcmkwin/kwintabbox/qml/main.qml. The main.qml cannot be installed becuase it uses some Q_PROPERTY elements. Any ideas about how to fix that?
> 3)I think that the starting comments of the tabbox should become better. I would prefer something like the "mainPlasmoid.qml"....
> 
> 
> Diffs
> -----
> 
>   templates/CMakeLists.txt 6a82772 
>   templates/mainTabbox.qml PRE-CREATION 
>   startpage.h 0df4c21 
>   startpage.cpp 9774b48 
>   editors/metadata/metadataeditor.cpp fce65fd 
>   editors/editpage.cpp a331ae5 
> 
> Diff: http://git.reviewboard.kde.org/r/105011/diff/
> 
> 
> Testing
> -------
> 
> 
> Screenshots
> -----------
> 
> 
>   http://git.reviewboard.kde.org/r/105011/s/574/
> 
> 
> Thanks,
> 
> Antonis Tsiapaliokas
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20120524/16b34d79/attachment.html>


More information about the Plasma-devel mailing list