noctalia-shell icon indicating copy to clipboard operation
noctalia-shell copied to clipboard

KeyboardLayout: subscribe to Sway events to update keyboard layout

Open 3akev opened this issue 1 month ago • 3 comments

Pull Request

Motivation

Following up on https://github.com/noctalia-dev/noctalia-shell/pull/706, now that https://github.com/quickshell-mirror/quickshell/pull/356 is merged, this receives Sway IPC events in order to update the keyboard layout in real time.

This should probably wait until the underlying API is available in a stable Quickshell release.

Type of Change

Mark the relevant option with an "x".

  • [ ] Bug fix
  • [ ] New feature
  • [ ] Breaking change
  • [x] Refactoring

Testing

Describe how you tested your changes and mark the relevant items.

  • [ ] Tested on niri
  • [ ] Tested on Hyprland
  • [x] Tested on sway
  • [ ] Tested with different bar positions and density settings
  • [ ] Tested at different interface scaling values
  • [ ] Tested with multiple monitors (if applicable)

Checklist

  • [x] Code follows project style guidelines
  • [x] Self-reviewed my code
  • [x] No new warnings or errors
  • [ ] Documentation or comments updated (if relevant)

Additional Notes

Add any additional context or follow-up notes for reviewers.

3akev avatar Nov 22 '25 23:11 3akev

Hey there @3akev , I'm just wondering if you plan to keep going with this PR or not :).

Ly-sec avatar Nov 29 '25 07:11 Ly-sec

This PR is ready, but it uses I3IpcListener which is a new QML component not yet available on the latest release of quickshell. Merging this now will probably break things for users not running the git build, so it's probably wise to hold off on merging this until quickshell's next release.

3akev avatar Nov 29 '25 11:11 3akev

This PR is ready, but it uses I3IpcListener which is a new QML component not yet available on the latest release of quickshell. Merging this now will probably break things for users not running the git build, so it's probably wise to hold off on merging this until quickshell's next release.

Ah thank you for letting me know :)! We'll wait until next QS release then. Thank you very much :heart: !

Ly-sec avatar Dec 01 '25 15:12 Ly-sec