Skip to content

feat(niri): niri depends on socket#4834

Open
khaneliman wants to merge 2 commits intoAlexays:masterfrom
khaneliman:niri
Open

feat(niri): niri depends on socket#4834
khaneliman wants to merge 2 commits intoAlexays:masterfrom
khaneliman:niri

Conversation

@khaneliman
Copy link
Contributor

@khaneliman khaneliman commented Feb 9, 2026

Don't attempt to use niri modules when socket connection fails. Prevents rendering modules when running another compositor. In same concept as previous Hyprland change 4295faa

Trying niri out and noticed this issue we previously had with Hyprland workspaces. When enabling multiple compositors in a same config, we should only render what's relevant / usable.
image

AFTER:

[2026-02-09 08:58:34.824] [warning] module niri/workspaces: Disabling module "niri/workspaces", Niri IPC: NIRI_SOCKET was not set! (Is Niri running?)
[2026-02-09 08:58:34.841] [warning] module sway/workspaces: Disabling module "sway/workspaces", Socket path is empty
[2026-02-09 08:58:34.843] [warning] module niri/window: Disabling module "niri/window", Niri IPC: NIRI_SOCKET was not set! (Is Niri running?)
[2026-02-09 08:58:34.858] [warning] module sway/window: Disabling module "sway/window", Socket path is empty

Properly disable the modules to not render blank spaces.

khaneliman added a commit to khaneliman/khanelinix that referenced this pull request Feb 9, 2026
Prevent invalid niri workspace module from rendering box. Remove after
upstreamed in Alexays/Waybar#4834
khaneliman added a commit to khaneliman/khanelinix that referenced this pull request Feb 9, 2026
Prevent invalid niri workspace module from rendering box. Remove after
upstreamed in Alexays/Waybar#4834
@khaneliman
Copy link
Contributor Author

@Alexays if you get a chance. just a QOL thing for people using multiple different compositors on the same machine

Don't attempt to use niri modules when socket connection fails. Prevents
rendering modules when running another compositor. In same concept as
previous Hyprland change
Alexays@4295faa

Signed-off-by: Austin Horstman <khaneliman12@gmail.com>
Move constructor from hpp to cpp to align with other modules

Signed-off-by: Austin Horstman <khaneliman12@gmail.com>
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.

1 participant