oamaok / kaiku Goto Github PK

Using JSX attribute disabled={true|false} does not work as one might intuitively expect, I tested that at least React handles this as one would expect. The correct behavior can still be achieved by passing disabled={undefined} instead of false which makes sense given how it behaves in HTML, so if you want to require users to do this in the name of library size maybe there should at least be a warning in development builds.

Reproducible on the playground with the following example:

export default function DisabledInput() {
  // The input should _not_ be disabled, but you cannot interact with it
  // Results in: <input type="input" disabled="false">
  return <input type="input" value="test" disabled={false} />
}

The following call count should stay at 7. The comment above is also wrong.

Also reference the obj, not the proxy. Faster access, as access to the proxy variable will always go through then handler.