mirror of
https://github.com/LGUG2Z/komorebi.git
synced 2026-01-11 14:40:25 +01:00
[BUG]: Toggle Pause Off not coming through in Named Pipe (to YASB) #342
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @jpmiller25 on GitHub (Apr 15, 2024).
Diagnosing some issues in the YASB integration, I found that it seems the TogglePause event is coming through on the named pipe only when toggling pause ON. When toggling pause OFF, there is no event, so YASB is not picking it up.
I'm completely lost in rust, just barely fluent in Python, so I'm not sure how to check from komorebi side. If yall can confirm the pipe is working from komorebi, just let me know I'll go back to messing with yasb :)
To Reproduce
I added a logger in the YASB event listener on the named pipe. I'm logging all buffer reads (win32pipe.PeekNamedPipe()) to make sure I'm getting the raw pipe, rather than anything after YASB code filters anything out. The event is not coming through when toggling Pause OFF.
Expected behavior
Pipe events when toggling pause on and off
Operating System
OS Name: Microsoft Windows 10 Business
OS Version: 10.0.19045 N/A Build 19045
komorebic checkOutputAdditional context
Running 0.1.24 release, latest main branch of darth/yasb.
@1096015776 commented on GitHub (May 11, 2024):
yes, i use your yasb , in python version, i can focus windows,then the active workspace can change.I read rust version,I found it not work when add KomorebiFocusNamedWorkspace listener
@LGUG2Z commented on GitHub (May 12, 2024):
6a1ed3bcaa (diff-704dbf9c754ba46b11956db31592bc56081bdbdcd69cac5b9c827a9d446866eeR1320)Fixed here, the untile event now gets sent properly 🎉