<p>Oh my gosh yes, I've always hated that feature. When the desktop is locked things shouldn't move around like that... I'd love to see that disappear. </p>
<div class="gmail_quote">On Sep 16, 2012 7:55 AM, "Marco Martin" <<a href="mailto:notmart@gmail.com">notmart@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Sunday 16 September 2012, Reza Shah wrote:<br>
> Hi,<br>
><br>
> I have pushed some changes for comic plasmoid at rshah/comicqml.<br>
> Basically i just move some UI parts to QML side, and left the logic as<br>
> current(hopefully)<br>
><br>
> As i'm never use this plasmoid before, i'm not sure if some bugs<br>
> introduced in this porting exist previously or new bugs.<br>
><br>
> There are some issues i'm aware off, and need some inputs:<br>
> - i'm not using(not ported yet) ImageWidget class, i just use normal<br>
> QImageItem (not sure if we really need scrollbars as implemented in<br>
> ImageWidget)<br>
<br>
is fine, keep it simple(tm)<br>
is one of the points of qml plasmoids.<br>
<br>
i think it should:<br>
* keep the image scaled keeping proportions, always scaled to display it full<br>
* top level window unscaled, with optional scrollbars triggered by middle<br>
mouse button and the 1:1 tool button<br>
<br>
> - not sure how to port the ability to change plasmoid size after comic<br>
> strips loaded (ComicApplet::updateSize()).<br>
<br>
i always very much disagreed with that feature, it did break it for any<br>
containment that is not the desktop.<br>
should be dropped and strongly discouraged in any plasmoid<br>
<br>
> - 'Actual Size' action not implemented, related to ImageWidget.<br>
> - The cursor for comic identifier and comic url label not changed to<br>
> pointing hand<br>
<br>
yeah, the hardest part will be the full size view by middle click<br>
> - Not sure if ComicTabbar (qml) should be exposed to C++ side or not.<br>
> - The buttons(prev,next,zoom) in ButtonBar, will not be highlighted on<br>
> mouse hover<br>
<br>
i seen that part seems to use a Rectangle, it should be a Toolbar with<br>
ToolButtons inside, i may take a look on that.<br>
<br>
Cheers,<br>
Marco Martin<br>
_______________________________________________<br>
Plasma-devel mailing list<br>
<a href="mailto:Plasma-devel@kde.org">Plasma-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/plasma-devel" target="_blank">https://mail.kde.org/mailman/listinfo/plasma-devel</a><br>
</blockquote></div>