John Jenkins

Results 120 comments of John Jenkins

I believe the repro just describes the expected / desired behaviour unless I'm missing something? For more context around the intention of this behaviour, [here's the original PR ](https://github.com/stenciljs/core/pull/6096)and [here's...

Hey! I think this should be fixed by https://github.com/stenciljs/core/pull/6187 Please let me know if you still experience any issues :)

hey @yigityuce (anyone else still interested) ... I personally think this is 1/2 a bug and 1/2 intended behaviour :D The `dist` output is actually 2 outputs (with variants) in...

@trusktr what happens if you change the listener to`onCustomEvent` (with a capital 'C')? I think stencil, rightly or wrongly, checks for `on{UPPERCASELETTER}whatever` for event handlers.

https://codesandbox.io/p/devbox/custom-event-listener-n33n9r ![Image](https://github.com/user-attachments/assets/2480c5b4-8d40-4216-8cbd-ad3d16004ee1) I think it's working?

> Would the usage be like this then: > ```html > > ... > ``` Essentially, although there's 2 different examples mentioned. A general styled block: ```html ``` (^ perhaps...

Just a note to say `shamefully-hoist=true` (in `.npmrc` for example) is a workaround obvs

👋!! I tried running with https://www.npmjs.com/package/vite-plugin-commonjs but came up against require statements that were too dynamic to be statically analysed within some of the jest deps. I guess / hope...

I believe the fix for this lies around [this vite config](https://github.com/webdriverio/webdriverio/blob/a7f0c07b223c9a4a7774fead36ec82296a99177f/packages/wdio-browser-runner/src/vite/constants.ts#L48). PNPM doesn't place those modules within my `node_modules` (because it only installs *my* project's deps in there). I guess...

@christian-bromann atm this is adding the `` as a sibling to the incoming component: ```html ... ... ``` Do you foresee any issue with that? Or do you think it...