davidedmundson added inline comments. INLINE COMMENTS
> modifier_only_shortcut_test.cpp:281 > // currently caps lock is on > - // shift is ignored > + // shift still triggers > kwinApp()->platform()->keyboardKeyPressed(modifier, timestamp++); I don't get this change: This is the " // this test verifies that Capslock does not trigger the shift shortcut" part of the test. Shift has been released on line 276, so triggeredSpy shouldn't be going up. REPOSITORY R108 KWin REVISION DETAIL https://phabricator.kde.org/D4241 EMAIL PREFERENCES https://phabricator.kde.org/settings/panel/emailpreferences/ To: graesslin, #kwin, #plasma Cc: davidedmundson, plasma-devel, kwin, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas
