Bug 426122 - redundant OSD indication when switching keyboard layout with Policy other than Global
Summary: redundant OSD indication when switching keyboard layout with Policy other tha...
Status: RESOLVED FIXED
Alias: None
Product: kwin
Classification: Plasma
Component: input (show other bugs)
Version: git master
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Andrey
URL: https://invent.kde.org/plasma/kwin/-/...
Keywords:
Depends on:
Blocks: 418699
  Show dependency tree / graph
 
Reported: 2020-09-02 20:57 UTC by Andrey
Modified: 2020-09-29 14:20 UTC (History)
2 users (show)

See Also:
Latest Commit: https://invent.kde.org/plasma/kwin/commit/db202f5c265d552ca8c908c72c1a46f9270b0291
Version Fixed In:
butirsky: Wayland+
butirsky: X11-

Attachments
Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Andrey 2020-09-02 20:57:12 UTC 
SUMMARY
Described the case with Application switching Policy.

STEPS TO REPRODUCE
1. Set Application Policy for keyboard layout switching in Keyboard Settings. Make sure there are at least 2 layouts.
2. Switch layout in one app, note OSD notification pop up on the switching.
3. Focus another app with mouse.
4. Start switching to the first app, now using Alt+Tab key combination. While pressing Alt+Tab, hold down Alt for a while, then Tab. 

OBSERVED RESULT
On Alt, OSD notification popup immediately even before the app switch.

EXPECTED RESULT
We are not interested in OSD for the previous app on app switching.

SOFTWARE/OS VERSIONS
Operating System: Ubuntu 20.04
KDE Plasma Version: 5.19.80
KDE Frameworks Version: 5.74.0
Qt Version: 5.14.2
Comment 1 David Edmundson 2020-09-29 14:20:36 UTC 
Git commit db202f5c265d552ca8c908c72c1a46f9270b0291 by David Edmundson, on behalf of Andrey Butirsky.
Committed on 29/09/2020 at 14:20.
Pushed by davidedmundson into branch 'master'.

fix: redundant keyboard layout OSD notification

there is no need to display the OSD for window we are going to switch from

DIGEST:

M  +2    -0    keyboard_input.cpp

https://invent.kde.org/plasma/kwin/commit/db202f5c265d552ca8c908c72c1a46f9270b0291