<table><tr><td style="">loh.tar updated this revision to Diff 57354.<br />loh.tar retitled this revision from "[WIP] ViewConfig: Use new generic config interface " to "ViewConfig: Use new generic config interface ".<br />loh.tar edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-tj7rqyatxvhufvn/">(Show Details)</a><br />loh.tar edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-jmdmyx4yhfr2jv2/">(Show Details)</a><br />loh.tar added a comment.
</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/D20924">View Revision</a></tr></table><br /><div><div><ul class="remarkup-list">
<li class="remarkup-list-item">Last missed settings</li>
<li class="remarkup-list-item">Some cleanup</li>
</ul></div></div><br /><div><strong>CHANGES TO REVISION SUMMARY</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Done so far:<br />
<br />
- DynWordWrapAlignIndent <br />
- DynWordWrapIndicators <br />
- DynWrapAtStaticMarker <br />
- DynamicWordWrap</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">This is a somehow raw transition, autotests are adapted too and still working</span> <br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- ShowWordCount</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">but there may still some adjustments needed, like value validators, fix</span> <br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- MaxHistorySize<br />
- SearchFlags</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">accidentally wrong converted default settings or whatever else.<br />
<br />
These two changes may cause some incompatibilities, too bad, but very nice</span> <br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- BackspaceRemoveComposedCharacters<br />
- ShowScrollBarPreview<br />
- ShowScrollBarMarks<br />
- ShowLineNumbers<br />
- ShowScrollBarMiniMap<br />
- ShowScrollBarMiniMapAll<br />
- ScrollBarMiniMapWidth<br />
- ShowIconBar<br />
- ShowScrollbars<br />
- ShowFoldingPreview<br />
- ShowLineModification<br />
- ShowFoldingBar<br />
- BookmarkSorting<br />
- AutoCenterLines<br />
- DefaultMarkTyp</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">in the long term:<br />
<br />
- Use the power of new the interface in ViewPrivate::setConfigValu</span>e<br />
- <span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">PersistentSelection<br />
- InputMode</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Use the power of new the interface in DocumentPrivate::setViewVariable<br />
<br />
With that we will have soon everywhere the same keys and the possibility</span> <br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- ViInputModeStealKeys<br />
- ViRelativeLineNumbers<br />
- AutomaticCompletionInvocation<br />
- WordCompletion<br />
- KeywordCompletion</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">literally to set each setting by document variables or the F7 command line</span> <br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">- WordCompletionMinimalWordLength<br />
- WordCompletionRemoveTail <br />
<br />
These two changes may cause some incompatibilities, too bad, but very nice in the long term<br />
<br />
- Use the power of new the interface in ViewPrivate::setConfigValue<br />
- Use the power of new the interface in DocumentPrivate::setViewVariable</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">without extra needed work for new options in the future.</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;"><span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">Please take a look, point me to mistakes, but wait until WIP is gone for a final "OK"</span></div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D20924?vs=57346&id=57354">https://phabricator.kde.org/D20924?vs=57346&id=57354</a></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D20924">https://phabricator.kde.org/D20924</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/src/katedocument_test.cpp<br />
autotests/src/kateview_test.cpp<br />
autotests/src/testutils.cpp<br />
autotests/src/vimode/base.cpp<br />
autotests/src/vimode/completion.cpp<br />
autotests/src/vimode/emulatedcommandbarsetupandteardown.cpp<br />
autotests/src/vimode/keys.cpp<br />
autotests/src/vimode/view.cpp<br />
src/dialogs/katedialogs.cpp<br />
src/document/katedocument.cpp<br />
src/search/katesearchbar.cpp<br />
src/utils/kateconfig.cpp<br />
src/utils/kateconfig.h<br />
src/view/katestatusbar.cpp<br />
src/view/kateview.cpp<br />
src/view/kateviewhelpers.cpp<br />
src/vimode/config/configtab.cpp</div></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, dhaumann<br /><strong>Cc: </strong>dhaumann, kwrite-devel, kde-frameworks-devel, KTextEditor, domson, michaelh, ngraham, bruns, demsking, cullmann, sars<br /></div>