Review Request 122924: Workaround bug with "per window" keyboard layout switching

Martin Klapetek martin.klapetek at gmail.com
Thu Dec 10 20:52:33 UTC 2015


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


As an update, I've made a proper patch fixing this at https://git.reviewboard.kde.org/r/126287/

But adding Martin G made him look for the cause of the bug, which is likely in Qt and will even
more likely get a fix elsewhere.

- Martin Klapetek


On March 12, 2015, 8:13 p.m., Eugene Shalygin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/122924/
> -----------------------------------------------------------
> 
> (Updated March 12, 2015, 8:13 p.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Bugs: 333742
>     http://bugs.kde.org/show_bug.cgi?id=333742
> 
> 
> Repository: ktp-text-ui
> 
> 
> Description
> -------
> 
> This must have be a bug in KWin. I believe it is sending window activation events when it shall not. As a result, layout switch is requested when another application is active and affects it, and this is very annoing. So, here we workaround this by waiting 200 ms and then checking is the chat windows still active.
> 
> 
> Diffs
> -----
> 
>   app/chat-window.h 0d1e0bb 
>   app/chat-window.cpp de07445 
> 
> Diff: https://git.reviewboard.kde.org/r/122924/diff/
> 
> 
> Testing
> -------
> 
> Manual.
> 
> 
> Thanks,
> 
> Eugene Shalygin
> 
>

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


More information about the KDE-Telepathy mailing list