elementary / wingpanel-indicator-keyboard

Wingpanel Keyboard Indicator
GNU General Public License v3.0
10 stars 7 forks source link

Switch engines using keyboard shortcut doesn't update its icon. #109

Open SteveFour opened 3 years ago

SteveFour commented 3 years ago

What Happened

When using multiple keyboard input method Default shortcut Ctrl + Space does change input method (switch engine), but does not change the Input method indicator icon on top right (the "en" thing in the image).

image while switching engines inside Settings app or Wingpanel widget works as usual.

Expected Behavior

Default shortcut Ctrl + Space should also change the Input method indicator on top right. That way people will know if they are using the right engine for the right language.

Steps to Reproduce

  1. Enable Ibus Daemon and add a second input method.
  2. Close System Settings app, then heads to any text field (Code app for example)
  3. Using shortcut to switch engine (Default Ctrl + Space) and try typing. Notice that input method did change but not the top right indicator.

Logs

Platform Information

image Latest version of Elementary OS 6 with all app updated (Aug 11 2021).

jeremypw commented 3 years ago

Note: the "Change Layout" shortcut (<Shift><Alt>) does change the indicator and cycles through both keyboard layouts and input methods. Not sure if this is the desired behaviour as, if more than one keyboard layout is installed, then the input methods use the last one selected which may not be the one wanted. "Change Layout" should only cycle through the available layouts. I think this is a consequence of both layouts and input methods being treated as "input sources" and "Change Layout" currently cycles through all input sources.