----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/127843/ -----------------------------------------------------------
(Updated Kvě. 26, 2016, 10:06 dop.) Review request for Kate, KDE Frameworks, Christoph Cullmann, and Dominik Haumann. Changes ------- Updated original version of this patch + added unit test. Is this test enough or you want more sofisticated one? Repository: ktexteditor Description ------- When using Indic locales composed characters are not properly removed. Pressing "delete" or "backspace" should remove the entire composed character and not only one base character. You can see how it should behave when using another text editor (e.g. libreoffice) Can be tested with these words: ?????????? or ??????????? or ???????? Technical details: I'm not really sure whether exporting current text layout is the right way to do this, I found that this is used when calling moveChar() which moves the cursor exactly by one composed character so I was trying to find a way to do it similarly. Diffs (updated) ----- autotests/src/katedocument_test.h 8abbad9 autotests/src/katedocument_test.cpp dd41e0f src/document/katedocument.cpp 73778a1 src/view/kateview.h 08db0df src/view/kateview.cpp fda6b2d Diff: https://git.reviewboard.kde.org/r/127843/diff/ Testing ------- Thanks, Jan Grulich
_______________________________________________ Kde-frameworks-devel mailing list Kde-frameworks-devel@kde.org https://mail.kde.org/mailman/listinfo/kde-frameworks-devel
