<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/116878/">https://git.reviewboard.kde.org/r/116878/</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;">That works around the problem, because the different initialization logic results in a different order of event filters installed on MarbleWidget: MarbleWidgetInputHandler will not install the event filters anymore in its ctor, but during installPluginEventFilter which is triggered by the layer manager initializing the plugin on its first render attempt. The (simplified) event filter order is then
without the patch: MarbleWidgetDefaultInputHandler => ScaleBar, OverviewMap
with the patch: ScaleBar, OverviewMap => MarbleWidgetDefaultInputHandler
The plugin's eventFilter method is then called before MarbleWidgetDefaultInputHandler does eat the MouseButtonRelease event, which prevents the bug. I'd rather have the original problem solved though, MarbleWidgetDefaultInputHandler should behave in a sane way and not require workarounds.
The following fixes the problem here:
git show -R c4e0678 | sed 's@src/lib/@src/lib/marble/@' | patch -p1
See https://bugs.kde.org/show_bug.cgi?id=242414 as well. The original bug (oxygen doing silly things) does not seem to occur anymore here, so reverting c4e0678 seems ok from my point of view. I'd like to have some other people test it before though.
Everyone, please give the following a try:
# change directory to marble sources
git show -R c4e0678 | sed 's@src/lib/@src/lib/marble/@' | patch -p1
# compile and run marble
# test that dragging the globe does not move the whole window as described in https://bugs.kde.org/show_bug.cgi?id=242414
</pre>
<br />
<p>- Dennis Nienhüser</p>
<br />
<p>On March 18th, 2014, 5:54 p.m. UTC, Abhinav Gangwar 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.</div>
<div>By Abhinav Gangwar.</div>
<p style="color: grey;"><i>Updated March 18, 2014, 5:54 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;">Now the info boxes (scale bar and map overview) doesn't follow the mouse.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Produces expected results on my system</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.h <span style="color: grey">(4a0f922)</span></li>
<li>src/plugins/render/mapscale/MapScaleFloatItem.cpp <span style="color: grey">(ea869bb)</span></li>
<li>src/plugins/render/overviewmap/OverviewMap.h <span style="color: grey">(4c9ada8)</span></li>
<li>src/plugins/render/overviewmap/OverviewMap.cpp <span style="color: grey">(ee52f03)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/116878/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>