Ben Grynhaus

Results 49 comments of Ben Grynhaus

Thanks @mskelton. Looks like it solves the first example I gave, but changing the setting to: ```json "tailwindCSS.experimental.classRegex": [ ["cx\\((([^()]*|\\([^()]*\\))*)\\)"], ["cva\\((([^()]*|\\([^()]*\\))*)\\)"], ] ``` Does seem to _sometimes_ work, but even...

Thanks @thecrypticace. If I want to have intellisense in multiple functions (e.g. `cx`, `cva`, `clsx`) - I need to repeat these 3 patterns for each, right? e.g. ```jsonc { "tailwindCSS.experimental.classRegex":...

> actually you could probably use a non-capturing group and one pattern. So replace `cva` with this `(?:clsx|cva|cx)` That's a great idea. Thanks! It looks good enough for now, even...

> Extension icons have been a bit of a challenge to get right lately. So thank you for filing this issue and investigating already. I'll make sure our dev team...

> It would be nice if each and every panel had its own icon. This would help make things look consistent, and some people like to use icons to quickly...

I've addressed this in https://github.com/DefinitelyTyped/DefinitelyTyped/pull/56102. The library could also be migrated to TypeScript, or include the type definitions within the package, but this is a good first step at least,...

> @bengry Nice! > > I, personally, strongly feel that types should be directly included in published repos. Why force developers to download/add a second dependency? (This is rhetorical) >...

> Thanks for tackling this, @bengry! > > I think this implementation might introduce a breaking change, as some folks could expect the `anchorElement` state in the popover store to...

> Anyway, IMO, especially because the library is still in pre-v1, and a migration guide is added to the release notes (basically, use [my workaround](https://github.com/ariakit/ariakit/issues/3729#issue-2259012011), but the other way around,...

Thanks @diegohaz. Works like a charm. ~Is `data-initial` documented anywhere?~ Nevermind, this is something that the consumer does. Could've been called whatever, since it's just binding the `initialRender` state to...