I submitted my patch here ( <a href="https://bugs.kde.org/show_bug.cgi?id=235545">https://bugs.kde.org/show_bug.cgi?id=235545</a>   ). Let me know If it requires more changes or not.<br><br>-Jignesh<br><br><div class="gmail_quote">
On Mon, Sep 26, 2011 at 3:32 PM, Aaron J. Seigo <span dir="ltr"><<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Sunday, September 25, 2011 18:39:10 jignesh kakadiya wrote:<br>
> <a href="http://paste.kde.org/126985/" target="_blank">http://paste.kde.org/126985/</a> here is the Ksnapshottimer.cpp<br>
<br>
</div>please post patches, not full copies of the source files, so that we can see<br>
the changes. `git diff` makes this easy.<br>
<br>
some comments:<br>
<br>
* please keep indentation and whitespace identical in the file to other uses.<br>
e.g. it should be "if (" not "if(" and the opening curly brace on line 84<br>
looks like it is misalinged<br>
<br>
* an implementation of enterEvent would probably have been enough;<br>
mouseMoveEvent is only really better if one assumes that after moving the<br>
lable, it might still be under the mouse ... but in that case moving it back<br>
will have the same problem. enterEvent allows you to avoid the check for "is<br>
in the mouse inside the widget" and so keeps the code simpler<br>
<br>
* you don't need two booleans for "onLeftCorner" and "onRightCorner". just one<br>
is enough. even then, a simple check for "if (x() == screenGeom.left())" would<br>
probably be enough to know if it is in the left corner.<br>
<br>
* the move call to (0, 0) on line 85 will break for screens whose geometry<br>
does not start at (0, 0). it should instead be move(screenGeom.topLeft()).<br>
same for the move call on line 78 which uses 0 for the y coordinate.<br>
<div class="im"><br>
> > /home/jiggy/kde/src/ksnapshot/snapshottimer.cpp:76:49: error: invalid use<br>
</div>> > of incomplete type â€˜struct QDesktopWidget’<br>
<br>
the file is missing: #include <QDesktopWidget><br>
<font color="#888888"><br>
--<br>
</font><div><div></div><div class="h5">Aaron J. Seigo<br>
humru othro a kohnu se<br>
GPG Fingerprint: 8B8B 2209 0C6F 7C47 B1EA  EE75 D6B7 2EB1 A7F1 DB43<br>
<br>
KDE core developer sponsored by Qt Development Frameworks<br>
</div></div><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>
<br></blockquote></div><br>