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

need clarity on combining .talon context header requirements #126

Open
codecat555 opened this issue Jun 11, 2021 · 1 comment
Open

need clarity on combining .talon context header requirements #126

codecat555 opened this issue Jun 11, 2021 · 1 comment

Comments

@codecat555
Copy link

In this section - https://talon.wiki/unofficial_talon_docs/#context-header - we have the following text:

Each kind of requirement can be listed several times. Entries of the same kind of requirement are OR‘d together, and of different kinds are AND‘d.

The 'same kind' language is ambiguous. Here's some alternate language (from aegis) which could be used instead:

talon ORs between the same key, because it needs to (you can't generally match app: slack and app: safari at the same time for example)
talon ANDs between different keys, because it needs to (app: slack, title: /talon/)

See https://talonvoice.slack.com/archives/C9MHQ4AGP/p1623385104250700 .

@codecat555
Copy link
Author

There is some back-and-forth discussion at the link referenced above, which may help in editing this section. Here, at the end, is another good summary statement: https://talonvoice.slack.com/archives/C9MHQ4AGP/p1623387488266100 .

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

No branches or pull requests

1 participant