Review Request 117729: Text-UI breaks "Per window" keyboard layout switching

Ahmed Ibrahim ahmedibrahimkhali at gmail.com
Wed Apr 23 19:46:06 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/117729/
-----------------------------------------------------------

(Updated April 23, 2014, 9:46 p.m.)


Review request for Telepathy.


Changes
-------

Fixed indentation issue.


Bugs: 333742
    http://bugs.kde.org/show_bug.cgi?id=333742


Repository: ktp-text-ui


Description
-------

This bug happens when the user has switched the keyboard layout of a chat tab and then switched to another application with another language, And he received a message or an "is typing" chat status, the language is then changed to the language of the Chat Tab that he was previously in, which is of course is an annoying behavior.

The reason of this behavior is that the restoreKeyboardLayout method is called inside the onCurrentIndexChanged without making sure that the Chat Window is active.


Diffs (updated)
-----

  app/chat-window.cpp a7da574 

Diff: https://git.reviewboard.kde.org/r/117729/diff/


Testing
-------

Changed language of the Chat Tab and then went to chromium and sent a message to myself from another account, that language did not change. 
Also tried composing a message to myself from another account and the language did not change.


Thanks,

Ahmed Ibrahim

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20140423/0e4aa902/attachment.html>


More information about the KDE-Telepathy mailing list