Fluentui: Improvment for react-hook `useBoolean`

[FuJuntao]

https://github.com/microsoft/fluentui/blob/3346b09e82b41f6b5b96fe1a94c8d122b3a4625b/packages/react-hooks/src/useBoolean.ts#L21-L38

We have a ref to store the boolean value, but what if we write it this way :

export function useBoolean(initialState: boolean): [boolean, IUseBooleanCallbacks] {
  const [value, setValue] = React.useState(initialState);

  const setTrue = useConst(() => () => {
    setValue(true);
  });
  const setFalse = useConst(() => () => {
    setValue(false);
  });
  const toggle = useConst(() => () => {
    setValue((value) => !value)
  });

  return [value, { setTrue, setFalse, toggle }];
}

It ensures toggle has a constant reference, and without having to store boolean value in a ref.

[paulgildea]

@FuJuntao Thanks for filing this. Is this something you'd be willing to contribute?

@tomi-msft - Coordinate with @FuJuntao if this is something they want to spin up a PR for.

[paulgildea]

@JustSlone / @dzearing FYI

[dzearing]

This is a simple improvement, feel free to submit a PR.

Minor thought, but just my 2c. I would love if the toggle function output took an optional argument to force it to a value. Call toggle with no arguments to toggle, or toggle(true) to force it to a state. That way we don't have to fumble with multiple functions.

[FuJuntao]

Thank you all for the replies, a PR is coming right away

[FuJuntao]

I have submitted the PR (#15607), but it seems that I'm not allowed to link it with this issue.

Anyway, it's ready for some reviews 💖

[msft-github-bot]

:tada:This issue was addressed in #15607, which has now been successfully released as @fluentui/[email protected].:tada:

Handy links:

• Release Notes
• Official NPM Release