Skip to content

KeyboardLayout: subscribe to Sway events to update keyboard layout#833

Merged
ItsLemmy merged 1 commit intonoctalia-dev:mainfrom
3akev:main
Mar 1, 2026
Merged

KeyboardLayout: subscribe to Sway events to update keyboard layout#833
ItsLemmy merged 1 commit intonoctalia-dev:mainfrom
3akev:main

Conversation

@3akev
Copy link
Copy Markdown
Contributor

@3akev 3akev commented Nov 22, 2025

Pull Request

Motivation

Following up on #706, now that quickshell-mirror/quickshell#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
  • Refactoring

Testing

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

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

Checklist

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

Additional Notes

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

@Ly-sec
Copy link
Copy Markdown
Collaborator

Ly-sec commented Nov 29, 2025

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

@3akev
Copy link
Copy Markdown
Contributor Author

3akev commented Nov 29, 2025

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.

@Ly-sec
Copy link
Copy Markdown
Collaborator

Ly-sec commented Dec 1, 2025

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 ❤️ !

@ItsLemmy
Copy link
Copy Markdown
Collaborator

ItsLemmy commented Feb 28, 2026

@3akev we are now running on noctalia-qs which is a pimped version of quickshell-git so in theory everyone should have access to I3IpcListener (have not checked tho).

Do you want to move ahead with this then?

@3akev
Copy link
Copy Markdown
Contributor Author

3akev commented Mar 1, 2026

Rebased and retested. Looks like it's good to go.

@3akev 3akev marked this pull request as ready for review March 1, 2026 10:51
@ItsLemmy ItsLemmy merged commit 1e56f67 into noctalia-dev:main Mar 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants