<table><tr><td style="">sars requested changes to this revision.<br />sars added a comment.<br />This revision now requires changes to proceed.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14620">View Revision</a></tr></table><br /><div><div><p>Hi,</p>

<p>I was just about to commit this change when I noticed that:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">The "Expand Tree" option is enabled at startup even if tree-mode is not enabled.</li>
<li class="remarkup-list-item">If I have "Expand Tree" disabled, no active symbol, a one tree expanded and I toggle "Show Parameters" it will collapse the tree.</li>
<li class="remarkup-list-item">Toggling off "Expand Tree" does not collapse all trees. It seems it first collapses all and then expands the tree of the current item. Actually I noticed that this patch does not introduce this "feature".</li>
</ol>

<p>Can you fix at least 1)?</p>

<ol class="remarkup-list" start="2">
<li class="remarkup-list-item">and 3) might require a bit more changes to get them to work logically and this patch is not introducing the problem.</li>
</ol>

<p>Regards,</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">Kåre</pre></div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14620">https://phabricator.kde.org/D14620</a></div></div><br /><div><strong>To: </strong>loh.tar, Kate, sars<br /><strong>Cc: </strong>sars, kwrite-devel, Kate, michaelh, kevinapavew, ngraham, demsking, cullmann, dhaumann<br /></div>