<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://svn.reviewboard.kde.org/r/1597/">http://svn.reviewboard.kde.org/r/1597/</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;">Are there any news on this?
The patch does not apply cleanly on trunk anymore.</pre>
<br />
<p>- Frederik</p>
<br />
<p>On September 29th, 2009, 10:27 p.m., Markus wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.reviewboard.kde.orgrb/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 Markus.</div>
<p style="color: grey;"><i>Updated 2009-09-29 22:27:55</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 adds a couple new features and wishlist items to knetwalk.
Overview:
1. auto-locking of pieces after they are rotated
2. auto-locking of adjacent pieces if you lock a piece
3. changing of cable animation time (wish/bug #186483)
4. advanced input mode, using Mouse and Keyboard together
5. board shifting in all modes where the connections wrap around the board
The changes in detail:
- added a page to the settings dialog where you can check if you want all these
things (they are all configurable independently of each other), and removed the
"Use Keyboard" entry in the menu
- changed the animation algorithm of the cells so as to be compatible with the
implementation of feature #2 (no visual difference, and a reduction in code
size)
- added a lock() method which checks for feature #2 and helps in implementation
of feature #1
- feature #1: after you have rotated a cell (i.e. when you change the next
cell) it will be locked automatically. This counts as a normal lock (i.e. as if
you had clicked with the mouse or used the keyboard), and so is 100% compatible
with feature #2
- feature #2: if a cell is locked, the adjacent cells will be checked if they
can be surely locked, i.e. there is no other position where they can fit with
the already locked cells (p.ex. a vertical bar next to another which is already
locked). This helps with the problem of Parker Coates (wish/bug #167588)
where cells are not locked because they are never rotated.
- feature #3: added an option to the settings dialog to change the animation
time (from 0 to 1000 ms). For this I had to change some details of the cell
implementation.
- feature #4: The settings page has an option to play with keyboard
alone (as it has been until now), or to use a combined mode in which a cell is
selected as the mouse hovers over it, and you can use the mouse and the
keyboard to rotate or lock it. This is the original mouse mode, but now with
some keyboard controls enabled.
- feature #5: When connections can be made around the edges of the board
(Expert mode), the board can be shifted by keyboard to make it easier to see the
missing links.
</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;">I fixed three bugs compared to the original version of the patch as published some time ago on bugs.kde.org, and now everything seems to work fine.
Note that I haven't split the patch as suggested on bugs.kde.org, because some features are inter-depending. I will remove all unwanted features from the patch once this has been reviewed.</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="https://bugs.kde.org/show_bug.cgi?id=167588">167588</a>,
<a href="https://bugs.kde.org/show_bug.cgi?id=186483">186483</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>/trunk/KDE/kdegames/knetwalk/src/settings.kcfgc <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/settings.ui <span style="color: grey">(PRE-CREATION)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/cell.cpp <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/globals.h <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/knetwalk.kcfg <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/knetwalkui.rc <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/mainwindow.h <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/mainwindow.cpp <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/cell.h <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/CMakeLists.txt <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/abstractgrid.h <span style="color: grey">(1014066)</span></li>
<li>/trunk/KDE/kdegames/knetwalk/src/abstractgrid.cpp <span style="color: grey">(1014066)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/1597/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>