[internal] Move `enabled` parameter in hooks to first argument

This PR is an internal refactor change that cleans up the signature of hooks that require an enabled state.

Whenever a hook requires an enabled state, the enabled parameter is moved to the front. Initially this was the last argument and enabled by default but everywhere that we use these hooks we have to pass a dedicated boolean anyway.

This makes sure these hooks follow a similar pattern. Bonus points because Prettier can now improve formatting the usage of these hooks. The reason why is because there is no additional argument after the potential last callback.

Before:

let enabled = data.__demoMode ? false : modal && data.comboboxState === ComboboxState.Open
useInertOthers(
  {
    allowed: useEvent(() => [
      data.inputRef.current,
      data.buttonRef.current,
      data.optionsRef.current,
    ]),
  },
  enabled
)

After:

let enabled = data.__demoMode ? false : modal && data.comboboxState === ComboboxState.Open
useInertOthers(enabled, {
  allowed: useEvent(() => [
    data.inputRef.current,
    data.buttonRef.current,
    data.optionsRef.current,
  ]),
})

Much better!

Name	Status	Preview	Comments	Updated (UTC)
headlessui-react	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 27, 2024 3:43pm
headlessui-vue	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 27, 2024 3:43pm

tailwindlabs / headlessui

[internal] Move `enabled` parameter in hooks to first argument #3245