PatrickJS

core: Can't assign "props" to "state"

8

we need a way to assign `props` to `state` https://github.com/BuilderIO/mitosis/blob/main/docs/gotchas.md#cant-assign-params-to-state we can either inject the workaround into onMount or for some generators

in most of the generated code this works as expected. https://github.com/BuilderIO/mitosis/blob/main/docs/gotchas.md#functions-cant-be-passed-by-reference-to-jsx-callbacks

We're destroying the view when a component updates to work around not having root nodes. We need a better ways to detect changes * We need to introduce a Proxy...

We should let the user overwrite what are attribute vs props or both with useMetadata

When there is a change to the jsx parser and other import parts used by Builder SDK we need to run the tests for Builder SDK with the changes made...

onUpdate first argument should be deps not second

2

the first arg should be the deps. We shouldn't follow react pattern for this since react was overloading useEffect for multiple usage (with dep array with no args, vs dep...

add useWindow useDocument

4

I think it's a good idea to add these hooks as each framework has their own way to manage these globals

certain elements may be the current activeElement. The list (from https://github.com/WICG/inert/blob/09c0647f59a3ad61ffbd92d17b9a0770fdbef412/src/inert.js#L24) * a[href] * area[href] * input:not([disabled]) * select:not([disabled]) * textarea:not([disabled]) * button:not([disabled]) * details * summary * iframe *...

setContext to pass a factory function

2

it would be great if we could pass a factory function to setContext ```jsx setContext(() => createInjector()); setContext(createInjector); setContext(() => { return { get: (token) => inject(token); }; }); ```

when we generate the code the helper functions such as `update` and `render` may conflict with any use defined method or prop. Later we may proxy the element for direct...