ipc-rules: Add get-view-pid to get the active view pid #2131

soreau · 2024-02-10T00:29:06Z

No description provided.

killown · 2024-02-10T04:47:20Z

could we have the pid info in "window-rules/get-focused-view" instead ?

so for example we could do things like [i["pid"] for i in list_view()] to get pid from all views

ammen99 · 2024-02-10T06:57:24Z

I think we don't need an additional method we should report this as a property of the view in the other methods like get-focused-view, list-views etc

plugins/single_plugins/ipc-rules.cpp

ammen99

LGTM, thanks!

I'll fix the tests and merge once I verify that everything works correctly :)

soreau force-pushed the ipc-get-view-pid branch from 6180b27 to 7acb8e3 Compare February 10, 2024 16:30

ammen99 reviewed Feb 10, 2024

View reviewed changes

plugins/single_plugins/ipc-rules.cpp Outdated Show resolved Hide resolved

soreau force-pushed the ipc-get-view-pid branch 4 times, most recently from 4f721c4 to d2733dd Compare February 10, 2024 16:56

ipc-rules: Add get-view-pid to get the active view pid

fd06c6f

soreau force-pushed the ipc-get-view-pid branch 2 times, most recently from 8d51819 to 5816ec2 Compare February 10, 2024 17:07

ipc-rules: Merge list_views() and view_to_json() properties

72a6886

soreau force-pushed the ipc-get-view-pid branch from 5816ec2 to 72a6886 Compare February 10, 2024 17:11

ammen99 approved these changes Feb 10, 2024

View reviewed changes

ipc-rules: Send view-unmapped signal

33c08d4

ammen99 merged commit 638a979 into master Feb 11, 2024
8 checks passed

ammen99 deleted the ipc-get-view-pid branch February 11, 2024 21:41

Provide feedback