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

[Commit Hook]

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


This review has been submitted with commit 68ffd6e4ec2525287a3a00ddbabefcac0be3ce9f by David Edmundson on behalf of Ahmed Ibrahim to branch master.

- Commit Hook


On April 23, 2014, 7:46 p.m., Ahmed Ibrahim wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/117729/
> -----------------------------------------------------------
> 
> (Updated April 23, 2014, 7:46 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> 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
> -----
> 
>   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/296c3744/attachment.html>