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

feat: allow writing custom text to stdin on button press #473

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

Conversation

atalii
Copy link

@atalii atalii commented Jun 13, 2023

This is just a small feature that I'll be using for my setup and notification daemon; you can merge it if you'd like :) Let me know if there's anything more to be done.

Here's the commit message explaining the change:

Rather than expecting external programs to handle 1, 2, and 3 on their standard input, this allows the user to configure the text sent themselves.

This is particularly relevant for my tool, painted, which serves as a notification daemon and takes commands to manage the notif queue over stdin. With this patch, a config like the following works just as one would expect:

[notifs]
command=painted
interval=persist

btn_left_cmd=clear
btn_right_cmd=expand

Note this only applies to bars with the persist interval, as one would expect.

Rather than expecting external programs to handle 1, 2, and 3 on
their standard input, this allows the user to configure the text
sent themselves.

This is particularly relevant for my tool, painted, which serves
as a notification daemon and takes commands to manage the notif
queue over stdin. With this patch, a config like the following
works just as one would expect:

	[notifs]
	command=painted
	interval=persist

	btn_left_cmd=clear
	btn_right_cmd=expand

Note this only applies to bars with the persist interval, as one
would expect.
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.

None yet

1 participant