<table><tr><td style="">cullmann 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/D17693">View Revision</a></tr></table><br /><div><div><p>I think I misunderstood the feature, sorry.</p>

<p>I did now try the patch again (it didn't apply perfectly, but that was just a few lines of code to be moved in the katedocument.cpp patch. I tend to agree that until we have perhaps some proper config for that in syntax files, some presets and the ability for users to add own presets make sense.</p>

<p>On can still later introduce some "syntax defined" enum value to query stuff from the syntax defintions.</p>

<p>What I would change is the placement of the SetOfCharsToEncloseSelection enum, I think it is only needed in the dialog, or? I got partly confused by that as I thought it would be used to store the settings, but the settings are just stored as stringlist and the enum is only to make the UI aware of the presets, or?</p></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D17693">https://phabricator.kde.org/D17693</a></div></div><br /><div><strong>To: </strong>loh.tar, KTextEditor, cullmann, mwolff, VDG<br /><strong>Cc: </strong>ngraham, mwolff, cullmann, sars, kwrite-devel, kde-frameworks-devel, KTextEditor, LeGast00n, domson, michaelh, bruns, demsking, dhaumann<br /></div>