D4135: Split implementation of keyboard layout handling into a dedicated class
Andrey Butirsky
noreply at phabricator.kde.org
Thu Dec 10 20:25:32 GMT 2020
bam added inline comments.
INLINE COMMENTS
> keyboard_input.cpp:709
> m_modifiersChangedSpy->updateModifiers(modifiers());
> + m_keyboardLayout->checkLayoutChange();
> }
I'm not quite follow the check here and whole processModifiers() method:
it's occurs after client gets the modifiers event, right?
But at this moment, the modifiers are already updated on server, isn't it?
Sorry I might miss something.
REPOSITORY
R108 KWin
REVISION DETAIL
https://phabricator.kde.org/D4135
To: graesslin, #plasma_on_wayland, #kwin, bshah
Cc: bam, bshah, plasma-devel, kwin, adam, retsim, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, hannahk, jraleigh, zachus, sgureev, fbampaloukas, mkulinski, ragreen, jackyalcine, schernikov, iodelay, crozbo, spoorun, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, eliasp, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kwin/attachments/20201210/d8b86d42/attachment.htm>
More information about the kwin
mailing list