D7660: Fix a regression caused by changing backspace key behavior

[Jan Grulich]

jgrulich added a comment.


  In https://phabricator.kde.org/D7660#144734, @safaalfulaij wrote:
  
  > In https://phabricator.kde.org/D7660#144706, @jgrulich wrote:
  >
  > > Not sure I can say much, I wrote this patch because I got a bug report where for Indic locales they were not able to remove composed characters. while other software (QtCreator, Libreoffice) worked pretty well. Also moving cursor to the left or to the right moved it by one composed character which I assumed should be same behaviour also for backspace/delete so I used same methods to identify one exact character which should be removed.
  >
  >
  > Well, I just checked both QtCreator and LibreOffice (using a text from Wikipedia) and both gave me different results than Kate currently doing. The results are the normal results that Arabic uses.
  >  I used them with thier defaults, so if there is any option hidden somewhere, then it's the same case I'm trying to achieve here.
  >  Where is that bug report?
  
  
  The bug report is not publicly available, but it was about kate not removing composed characters, while other software do that. I now checked this with LibreOffice and QtCreator and it seems to remove composed character only when you use DELETE and not BACKSPACE. Not sure what is correct, but having this as an option seems to be a good idea.

REPOSITORY
  R39 KTextEditor

REVISION DETAIL
  https://phabricator.kde.org/D7660

To: safaalfulaij, #ktexteditor, jgrulich, hein
Cc: jgrulich, dhaumann, hein, kwrite-devel, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwrite-devel/attachments/20170911/f1bbcfff/attachment-0001.html>