Vincent
Vincent
Moving this to the extensions repo btw
Oops indeed, totally forgot about `Vary`, you're totally right! Then I don't see why the extension should _not_ send the htmx headers, so if you'd like to suggest an implementation...
@frederikhors could you confirm if #106 (now merged) addresses all your concerns ? (new extension's version is not yet published, so you'd have to use the file directly from the...
Oh I see, then I'm closing this as it sounds like solved by #106 , feel free to reopen / ping if anyone encounters this again!
> I'm aware that this can probably be achieved with CSS animations, but I feel like a dedicated attribute works better. May I ask why? CSS sounds like the perfect...
Personal opinion: I don't really like this idea as it makes the code way harder to read imho, for what seems to bring not much value to me (except sparing...
> I also realize now this probably does not correctly handle the special global and host targets mentioned in the web component example. Should I look into that for this...
> I don't think it's currently implemented on the dev branch If it's not implemented at all, then I'd say you should indeed not care about it for _this_ PR...
Looking at the code, I see that `contextElement` is an optional param in the `swapOptions`, itself also being optional. We have a fallback for the `swapOptions` in the `swap` method...
Oof, good catch, hadn't realized this! Ok so it's hard to get `tsc` to generate what we want, we had to do crazy tricks sometimes to get a proper result...