<div dir="ltr"><br><br><div class="gmail_quote">2008/9/12 Aaron Seigo <span dir="ltr"><<a href="mailto:aseigo@kde.org">aseigo@kde.org</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d"><br>
-----------------------------------------------------------<br>
This is an automatically generated e-mail. To reply, visit:<br>
</div><a href="http://reviewboard.vidsolbach.de/r/181/#review190" target="_blank">http://reviewboard.vidsolbach.de/r/181/#review190</a><br>
-----------------------------------------------------------<br>
<br>
<br>
instead of returning a region that is used by resize areas .. what about a method that takes a point and returns a boolean on whether or not that point is in a control area of the dialog? this way it isn't biased towards resizing (we might add other features in the future) and we can avoid creating and testing with regions which often aren't the fastest things in the world.</blockquote>
<div><br>I totally agree with you :P<br>But.. can you suggest me a nice name for this method? maybe bool inControlArea(const QPoint &) ?<br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
- Aaron<br>
<div class="Ih2E3d"><br>
<br>
On 2008-09-09 05:17:19, Alessandro Diaferia wrote:<br>
><br>
> -----------------------------------------------------------<br>
> This is an automatically generated e-mail. To reply, visit:<br>
> <a href="http://reviewboard.vidsolbach.de/r/181/" target="_blank">http://reviewboard.vidsolbach.de/r/181/</a><br>
> -----------------------------------------------------------<br>
><br>
</div>> (Updated 2008-09-09 05:17:19)<br>
<div><div></div><div class="Wj3C7c">><br>
><br>
> Review request for Plasma.<br>
><br>
><br>
> Summary<br>
> -------<br>
><br>
> I reimplemented mouse events of Plasma::Dialog for my Previewer applet to allow users move the dialog here and there on the screen. Since the dialog is also resizable i need to know which part of the Plasma::Dialog are supposed to catch mouse events in order to avoid movement even when the user just wants to resize the dialog. Because of this i added this small function to Plasma::Dialog. Please tell me if it is ok to commit.<br>
><br>
><br>
> Diffs<br>
> -----<br>
><br>
> trunk/KDE/kdebase/workspace/libs/plasma/dialog.h<br>
> trunk/KDE/kdebase/workspace/libs/plasma/dialog.cpp<br>
><br>
> Diff: <a href="http://reviewboard.vidsolbach.de/r/181/diff" target="_blank">http://reviewboard.vidsolbach.de/r/181/diff</a><br>
><br>
><br>
> Testing<br>
> -------<br>
><br>
><br>
> Thanks,<br>
><br>
> Alessandro<br>
><br>
><br>
<br>
</div></div></blockquote></div><br><br clear="all">Regards<br>-- <br>Alessandro Diaferia<br>
</div>