Benjamin Bock

I hope you had a great Christmas break @grantbevis. I'm still postponing my upgrade from the mid-2021 version I'm currently running. Do you have an estimate on your big refactoring?...

Thanks for coming back to this. No questions for usage so far. I guess next steps for the repo are something like this: ```bash git rm msmtp git rm ntfy...

You need at least also remove them from `code_attr_delims` if I understand https://github.com/slim-template/slim/blob/main/lib/slim/parser.rb#L81 correctly. But I used those delims too often, so I didn't try further.

Yeah, sorry. Maybe I'll give it another try later but no plans for now.

I couldn't stop myself and gave it another look: We'll need to change https://github.com/slim-template/slim/blob/main/lib/slim/parser.rb#L73 from @attr_shortcut_re = /\A(#{keys}+)((?:\p{Word}|-|\/\d+|:(\w|-)+)*)/ to include `[` and `]` but only if they're not used in...

So setting ``` Slim::Engine.options[:attr_list_delims] = { '{' => '}' } ``` and and changing Line 73 to ``` @attr_shortcut_re = /\A(#{keys}+)((?:\p{Word}|-|\[|\]|\(|\)|\/\d+|:(\w|-)+)*)/``` actually makes ```.foo-[a(b)].bar()``` compile to `````` and ```.foo-[a(b)].bar() {realarg="value"}...

Neither worked with the previous state. As we're basically all-in with before content and after content, I now converted it to a negative selection. This also already includes forbidding the...

You're right. I did some additional changes: Removed (duplicate) `` because in some tests they were used as delimiters which produced warnings. Needed to forbid `:` and `/` at the...

Running the tailwindcss tests worked fine for me. After cloning and `npm install` I could run the whole suite using `npm run test` or just that file using `jest tests/default-extractor.test.js`...

This issue still exists with 6.4.1004.