> On Dez. 9, 2015, 7:59 vorm., Martin Gräßlin wrote: > > The interesting question is why does the OSD trigger a WindowDeactivate. > > This needs investigation as otherwise we start to work around issues (like > > here) without fixing the root cause.
I tried setting WindowDoesNotAcceptFocus on the Osd Dialog but that didn't help. - Kai Uwe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/126287/#review89270 ----------------------------------------------------------- On Dez. 9, 2015, 5:03 vorm., Martin Klapetek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/126287/ > ----------------------------------------------------------- > > (Updated Dez. 9, 2015, 5:03 vorm.) > > > Review request for Telepathy and Martin Gräßlin. > > > Bugs: 356220 > http://bugs.kde.org/show_bug.cgi?id=356220 > > > Repository: ktp-text-ui > > > Description > ------- > > So it turns out that with text-ui's "remember layout per tab" feature > and Plasma's OSD, there's a problem when activating the global keyboard > layout change shortcut. It triggers the OSD and in turn that causes > QEvent::WindowDeactivate and QEvent::WindowActivate events in the app, > leading to text-ui immediately setting the old keyboard layout, making > it look like it's actualy blocking the changes. > > While checking KWindowSystem::activeWindowChanged I noticed that the > active window actually does not change. So I've moved setting the kbd > layout to KWindowSystem::activeWindowChanged slot rather than in the > event() handler. Seems to work perfectly. > > I think this is the same bug that is causing the hidden panel to raise > on OSD/notifications and the one that is showing the "plasma" task in > task manager for split second, these two: > > https://bugs.kde.org/show_bug.cgi?id=352278 > https://bugs.kde.org/show_bug.cgi?id=332024 > > > Diffs > ----- > > app/chat-window.h 0d1e0bb > app/chat-window.cpp fdde18d > > Diff: https://git.reviewboard.kde.org/r/126287/diff/ > > > Testing > ------- > > With "remember keyboard layout per tab" I can now successfully change > keyboard layout from text-ui and the keyboard layout is properly restored > when switching to the window. > > > Thanks, > > Martin Klapetek > >
_______________________________________________ KDE-Telepathy mailing list [email protected] https://mail.kde.org/mailman/listinfo/kde-telepathy
