<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/115622/">https://git.reviewboard.kde.org/r/115622/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hi, can you please elaborate why this fixes the bug? I don't see the connection really. I suspect the fix has unwanted side effects (it breaks the init logic).
What I suspect to cause the bug (or one of its incarnations) is when the mouse button release event is not properly caught by the float item because the mouse has left the float item. This can happen e.g. if you move a float item to one of the sides and keep moving (mouse button pressed) the mouse out of the window. Below is a possible fix for that. That fix isn't too sane though because it results in sluggish move behavior: move a float item fast and it stops moving.
diff --git a/src/lib/marble/graphicsview/ScreenGraphicsItem.cpp b/src/lib/marble/graphicsview/ScreenGraphicsItem.cpp
index fbacad9..87dcefa 100644
--- a/src/lib/marble/graphicsview/ScreenGraphicsItem.cpp
+++ b/src/lib/marble/graphicsview/ScreenGraphicsItem.cpp
@@ -110,6 +110,11 @@ bool ScreenGraphicsItem::eventFilter( QObject *object, QEvent *e )
|| e->type() == QEvent::MouseButtonRelease )
{
QMouseEvent *event = static_cast<QMouseEvent*>( e );
+ if ( !contains( event->pos() ) ) {
+ p()->m_floatItemMoving = false;
+ return false;
+ }
+
// The rect the item was painted on before. We add one pixel as antialiasing could
// result into painting on these pixels to.
QRectF floatItemRect = QRectF( positivePosition() - QPoint( 1, 1 ),
</pre>
<br />
<p>- Dennis Nienhüser</p>
<br />
<p>On February 12th, 2014, 12:39 p.m. UTC, Piotr Rżysko wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Marble, Dennis Nienhüser and Torsten Rahn.</div>
<div>By Piotr Rżysko.</div>
<p style="color: grey;"><i>Updated Feb. 12, 2014, 12:39 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=324862">324862</a>
</div>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
marble
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Scale bar and Overview Map don't follow mouse now.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/plugins/render/mapscale/MapScaleFloatItem.cpp <span style="color: grey">(df0bb57)</span></li>
<li>src/plugins/render/overviewmap/OverviewMap.cpp <span style="color: grey">(ceb56ba)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/115622/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>