Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Json chooser #105

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

columbarius
Copy link
Collaborator

Adds the chooser_type json, which calls the chooser with the output_list formated as json with a script parsing this object and calling a dmenu chooser.

Depends: #59

@emersion
Copy link
Owner

emersion commented Apr 2, 2021

A script can just get info from the Wayland connection. I don't think should do the work for them.

@emersion
Copy link
Owner

emersion commented Apr 2, 2021

Note to self: maybe it would be nice to add JSON output support to wayland-info for this kind of use-case.

(Ref https://gitlab.freedesktop.org/wayland/wayland-utils/-/issues/2)

@columbarius
Copy link
Collaborator Author

Note to self: maybe it would be nice to add JSON output support to wayland-info for this kind of use-case.

That was mainly the reason why I wrote this.
The other one was to test, how easy it would be to add another chooser_type and it showed that the switch trick no longer works.
Also this way we could do windows, when wlroots screencopy has support for that.

But feel free to close the MR if it's currently out of scope.

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.

2 participants