WIP: Implement more workspace events #70

Open
clktmr wants to merge 4 commits from clktmr/yambar:i3-workspace-rename into master
clktmr commented 5 months ago

A renamed workspace caused yambar to abort in a failed assertion,
because workspace lookup was done by name and the rename event was not
implemented. To resolve this issue this patch implements the rename
event and as a necessity changes workspace_lookup() to use ids instead
of names.

A renamed workspace caused yambar to abort in a failed assertion, because workspace lookup was done by name and the `rename` event was not implemented. To resolve this issue this patch implements the `rename` event and as a necessity changes workspace_lookup() to use ids instead of names.
clktmr added 1 commit 5 months ago
8b3919a8ea modules: Implement workspace rename event
clktmr added 3 commits 5 months ago
77183e6da0 modules: Implement workspace move event
clktmr changed title from Implement workspace rename event to WIP: Implement more workspace events 5 months ago
dnkl requested changes 5 months ago
dnkl left a comment

I've only had a brief look; things in general look good. Can you add a changelog entry (under "fixed"). And add yourself to the contributors list, if you'd like.

I hope to be able to take a closer look at the actual code soon.

Reviewers

dnkl requested changes 5 months ago
This pull request has changes conflicting with the target branch.
modules/i3.c
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.