<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/103619/">http://git.reviewboard.kde.org/r/103619/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On January 5th, 2012, 11:51 a.m., <b>Andreas Pakulat</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;">Current master requires KDE 4.6 already I believe (if not it should due to the huge amount of fixes related to kate).
I didn't look at the patch itself yet, but how does your code handle the case of two project views being restored? As far as I recall there's no API for plugins/views to find out which kconfig group to use to restore view-state from since this is completely hidden in sublime/shell code.
Apart from that IMHO storing this in the project configuration is wrong. The view state has nothing to do with the project itself, its about view-state and view-state should go into the session configuration file. The project-configuration file is for project-specific data, which expansion state is not, since I can load the same project into two different sessions and would not want one session to overwrite the view-state of another session for that project.</pre>
</blockquote>
<p>On January 5th, 2012, 7:32 p.m., <b>Andrew Fuller</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;">CMakeLists still has set(KDE_MIN_VERSION "4.5.0") I can bump that if you like, but that doesn't belong in this patch considering the remove of the #if/#else (below).
Re: multiple project views. Until now I was not aware that this was even possible. I will address that in my updated patch (which will be hopefully tomorrow).</pre>
</blockquote>
<p>On January 5th, 2012, 8:25 p.m., <b>Andreas Pakulat</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;">I'd say it should be bumped, but I'm not active anymore so I'll leave that to the active group of devs.
As I said, I don't think you'll be able to adress multiple project views currently since a view-object does not know which config group to read the state from. This has been a long-time todo for the sublime/shell API but didn't get done yet (again afaik, I'm a bit out of touch these days).
Anyway, if Milian is ok with that it might still be useful enough to warrant this minor issue, after all there are not going to be many people with more than 1 projects view. There's IIRC also a plan to have some views as 'singleton', i.e. only a single instance of the view is ever created and the project tree is a prime candidate for that IMHO.</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 tried adding a second project view in a session, but when I exited and re-opened the session I was presented with a single project view. The upside of this behaviour is that saving/restoring the state of the project view need not concern itself with multiple views!</pre>
<br />
<p>- Andrew</p>
<br />
<p>On January 7th, 2012, 2:45 a.m., Andrew Fuller 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 Andrew Fuller.</div>
<p style="color: grey;"><i>Updated Jan. 7, 2012, 2:45 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;">Save/Restore the ProjectTreeView. This enables a more seamless transition when returning to a project that was closed.</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;">Load a session, Exit, and return. Voilà! The tree is right where you left it.</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>plugins/projectmanagerview/CMakeLists.txt <span style="color: grey">(7b58e8e5489819b0370f5d55e0218f063da90af2)</span></li>
<li>plugins/projectmanagerview/projecttreeview.h <span style="color: grey">(16893f0713b83a78a5b025a55fcfa082265d4419)</span></li>
<li>plugins/projectmanagerview/projecttreeview.cpp <span style="color: grey">(8237af513627c0459d571de2bccc275e8a4b0ae9)</span></li>
<li>project/CMakeLists.txt <span style="color: grey">(bac4b22b8a9966f4efe4080a6d24e0de17c48e12)</span></li>
<li>project/projectmodelsaver.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>project/projectmodelsaver.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>shell/sessioncontroller.h <span style="color: grey">(551894d4131218b9ab2640e28f2c354ce319a105)</span></li>
<li>shell/sessioncontroller.cpp <span style="color: grey">(d2ee8732710e0e4bded56e547d83a3906b20b018)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/103619/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>