<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/103764/">http://git.reviewboard.kde.org/r/103764/</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;">I disagree, at least for the review-area. Maybe its ok if the review area is not needed anymore (i.e. the patch is comitted, no local changes anymore), but then the user would've ended up in the code area anyway before closing kdevelop.
Maybe for the debug area kdevelop should rather start the debugee(s) that where left running again?</pre>
<br />
<p>- Andreas</p>
<br />
<p>On January 22nd, 2012, 5:07 p.m., Niko Sams 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 KDevelop.</div>
<div>By Niko Sams.</div>
<p style="color: grey;"><i>Updated Jan. 22, 2012, 5:07 p.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;">Areas are used in a workflow:
- debugging
- commiting/patch reviewing
and kdevelop switches automatically to that area (and back)
Now when I have a running debug session (and debug area is opened) and I quit kdevelop, it will remember debug as active area and restore it on start.
That doesn't make much sense in the way areas are used now, so my patch removes that functionality.
...ok to go into 1.3?</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>shell/uicontroller.cpp <span style="color: grey">(2091fdf)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103764/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>