<div dir="ltr">Hey,<div><br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">


---------- Forwarded message ----------<br>
From: Conny Marco Menebröcker <<a href="mailto:admin@menebroecker-web.de">admin@menebroecker-web.de</a>><br>
Date: Sat, Feb 15, 2014 at 7:44 PM<br>
Subject: Request for review<br>
To: <a href="mailto:kde-devel@kde.org">kde-devel@kde.org</a><br>
<br>
<br>
Hello all,<br>
<br>
<br>
I have written my first plasmoid. I called it PlasmaTaskViewer,<br>
because it views the todos of a calendar file.<br>
In my case it shows the entries of my iPhone reminder app.<br>
<br>
I would be happy if someone reviews my code. I am especially<br>
interested in comments about the use of kcalcore and the data engine.<br>
But every other comment is welcome, too.<br></blockquote><div><br></div><div>I just did a quick look over your code and tried to build it. You'll need to explicitly look for KdepimLibs + set include_dirs in order to build everywhere with no problems. Attached is a simple patch fixing the build. </div>

<div> </div><div>I tried running it but have no .ics files present so can't really try it out. Though your configuration dialog is somewhat too small and cannot be resized easily for some reason. One more comment on the visuals - if I add the applet is added on the desktop and you have no data/must set it up first, it should show some label what to do, otherwise it's just an empty Plasma-themed rectangle sitting on your desktop :)</div>

<div><br></div><div>Final note, we do not really support Applets based on QGraphicsView anymore, the proper way to go is to use QML, which is way superior in tech and ease of writing. I think this[1] should be a good way to get you started :)</div>

<div><br></div><div>[1] - <a href="http://techbase.kde.org/Development/Tutorials/Plasma/QML/GettingStarted">http://techbase.kde.org/Development/Tutorials/Plasma/QML/GettingStarted</a></div><div><br></div></div><div>Cheers</div>

-- <br><div><span style="color:rgb(102,102,102)">Martin Klapetek | KDE Developer</span></div>
</div></div></div>