<table><tr><td style="">andreylegayev created this revision.<br />Herald added a project: KDE PIM.<br />Herald added a subscriber: kde-pim.<br />andreylegayev requested review of this revision.
</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/D28994">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Fixes applied:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">TabWidget and KeyTreeView were saving sorting preset to the same config group.  I've fixed it by setting config group to View #1, 2, 3... as it should be. I've also had to prevent saving empty filter of last tab.</li>
<li class="remarkup-list-item">TabWidget didn't save settings on exit - added call to saveViews() to destructor</li>
<li class="remarkup-list-item">Prevented saving empty filter for the last tab in TabWidget::setKeyFilter()</li>
</ol>

</div></div><br /><div><strong>TEST PLAN</strong><div><p>Steps:</p>

<ol class="remarkup-list">
<li class="remarkup-list-item">Create several tabs with different filters</li>
<li class="remarkup-list-item">Apply sorting to some columns</li>
<li class="remarkup-list-item">Exit from kleopatra</li>
<li class="remarkup-list-item">Run kleopatra</li>
</ol>

<p>Expected result: <br />
List of tabs, filters and sorting should be restored after app restart</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R168 Kleopatra</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28994">https://phabricator.kde.org/D28994</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/view/tabwidget.cpp</div></div></div><br /><div><strong>To: </strong>andreylegayev<br /><strong>Cc: </strong>kde-pim, fbampaloukas, dcaliste, dvasin, rodsevich, winterz, vkrause, mlaurent, knauss, dvratil<br /></div>