Skip to content

Add "short paint" modifier #587

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

Closed
pokey opened this issue Mar 3, 2022 · 3 comments · Fixed by #737
Closed

Add "short paint" modifier #587

pokey opened this issue Mar 3, 2022 · 3 comments · Fixed by #737
Labels
enhancement New feature or request

Comments

@pokey
Copy link
Member

pokey commented Mar 3, 2022

Defined as intersection of "paint" and "string", so for example:

foo("bar-baz bongo")

Saying "take small paint zip" would select bar-baz. This modifier will be really useful for tailwind.css

Once we have #69, we can implement this one using an intersection pipeline constructor, which will take n pipelines as its arguments, and construct a new pipeline stage that applies each of its pipeline arguments to its input, and outputs the intersection of the outputs of its pipelines

@pokey pokey added the enhancement New feature or request label Mar 3, 2022
@AndrewDant
Copy link
Collaborator

@pokey I think I could implement this one using the existing code. Do you think I should wait for your big PR anyway?

@pokey
Copy link
Member Author

pokey commented Jun 1, 2022

I would wait for the big PR, but you don't need to go the fancy generic intersection stage route. Hopefully big PR will be out tomorrow or next day. It's getting really close

@AndrewDant
Copy link
Collaborator

@pokey I wonder if we should expand this to other nonwhite space delimiters such as commas and parentheses. Would you support that idea or do you think it's redundant with other scope types?

@pokey pokey changed the title Add "small paint" modifier Add "short paint" modifier Jul 3, 2022
@pokey pokey closed this as completed in #737 Jul 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants