<table><tr><td style="">ahmadsamir updated this revision to Diff 64708.<br />ahmadsamir retitled this revision from "Enable typing soft-hyphen characters" to "Mimic QInputControl::isAcceptableInput() when filtering typed characters".<br />ahmadsamir edited the summary of this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-x3e7wenwoczfqlx/">(Show Details)</a><br />ahmadsamir edited the test plan for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-d7vjkdjaxhokwzt/">(Show Details)</a><br />ahmadsamir 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/D23472">View Revision</a></tr></table><br /><div><div><p>Rework the whole patch</p></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);">The typeChars() functions filters out non-printable characters, except for:</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">Move all input characters filtering out of typeChars() to KateViewInternal<br />
and filter the input before sending it to typeChars().<br />
<br />
This increases the scope of unicode characters that users can type in ktexteditor.<br />
For more info see:<br />
QChar documentation<br />
http://www.unicode.org/Public/UNIDATA/UnicodeData.txt<br />
<br />
This should fix:<br />
- bug 396764 (typing soft-hyphens)<br />
- bug 366424 (typing "private use" unicode characters)</span><br />
<span style="padding: 0 2px; color: #333333; background: rgba(251, 175, 175, .7);">\t, \n, \r; add QChar::SoftHyphen to that list.</span><span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">- Hopefully bug 389796 (typing formatting characters such as ZWNJ)</span><br />
<br />
BUG: 396764<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);"><br />
BUG: 366424<br />
BUG: 389796</span></div></div></div><br /><div><strong>CHANGES TO TEST PLAN</strong><div><div style="white-space: pre-wrap; color: #74777D;">Test typing a soft-hyphen char (here I used Compose key + minus + minus + space)<br />
<span style="padding: 0 2px; color: #333333; background: rgba(151, 234, 151, .6);">For the two other bugs, ask the users to test...<br />
<br />
</span>All unit tests passed</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R39 KTextEditor</div></div></div><br /><div><strong>CHANGES SINCE LAST UPDATE</strong><div><a href="https://phabricator.kde.org/D23472?vs=64693&id=64708">https://phabricator.kde.org/D23472?vs=64693&id=64708</a></div></div><br /><div><strong>BRANCH</strong><div><div>ahmad/soft-hyphen (branched from master)</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D23472">https://phabricator.kde.org/D23472</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>src/document/katedocument.cpp<br />
src/document/katedocument.h<br />
src/view/kateviewinternal.cpp<br />
src/view/kateviewinternal.h</div></div></div><br /><div><strong>To: </strong>ahmadsamir, KTextEditor, dhaumann, cullmann<br /><strong>Cc: </strong>kde-frameworks-devel, kwrite-devel, LeGast00n, GB_2, domson, michaelh, ngraham, bruns, demsking, cullmann, sars, dhaumann<br /></div>