<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="http://git.reviewboard.kde.org/r/107067/">http://git.reviewboard.kde.org/r/107067/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On December 13th, 2012, 7:03 p.m., <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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, this patch mixes bug fixes and features right? (We don't have anyone very much knowledgeable in kmines at the moment, so sorry if i'm doing silly questions)</pre>
</blockquote>
<p>On December 14th, 2012, 12:29 a.m., <b>Andrew Wallace</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<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 is correct. All but one line is to fix the bugs. I threw the left click acting like middle click in as a bonus, but if we are in the wrong window for features it is trivial to yank back out into a separate patch.
The difference I'm talking about is:
if(midButtonPressed || (ev->button() == Qt::LeftButton && itemUnderMouse && itemUnderMouse->isRevealed() ))
{
m_mouseAction = CLEAR_NEIGHBORS;
}
would be be only a bug fix if it were:
if(midButtonPressed)
{
m_mouseAction = CLEAR_NEIGHBORS;
}
</pre>
</blockquote>
<p>On December 14th, 2012, 5:43 p.m., <b>Albert Astals Cid</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">The thing is, we are on a feature freeze right now for 4.10 release in a few weeks, and can't have new features in there. So we can wait for 4.11 and get both or try to get it into 4.10 with just the bugfix. I probably prefer the second, could you retarget the diff to include only bugfixes and create a new review request with the new feature?
Thanks and sorry for the administrative overhead!</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I've removed the feature. I was unable to create a separate review for the feature, due to my lack of understanding of how to upload a patch that depends on this patch. In the case that this one is accepted I will recreate the review at that time.</pre>
<br />
<p>- Andrew</p>
<br />
<p>On December 15th, 2012, 1:53 a.m., Andrew Wallace wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Games.</div>
<div>By Andrew Wallace.</div>
<p style="color: grey;"><i>Updated Dec. 15, 2012, 1:53 a.m.</i></p>
<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;">This does two things:
1) Be safer about cleaning up visually 'pressing' squares.
2) Have a left click on a revealed square act the same as a middle click.
The patch changes how the mouse actions are handled. Storing what action is started with the mouse down event. Makes the logic more generic.
(Please be kind, it's my first patch.)</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;">Testing was by hand, here are the cases I made sure to try and looked for press artifacts:
1) Steps in bug 221725.
2) Middle drag then right clicked on tool bar.
3) Right down. Drag. Left down. Drag. Both release. (And vice versa.)
4) Middle down. Drag off play area. Release. Left click anything.</pre>
</td>
</tr>
</table>
<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=221725">221725</a>,
<a href="http://bugs.kde.org/show_bug.cgi?id=284952">284952</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>minefielditem.h <span style="color: grey">(a6319d3)</span></li>
<li>minefielditem.cpp <span style="color: grey">(3e77cc1)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/107067/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>