Check if user provided `toggle` is defined

[SimplyComplexable]

Currently this is done using a ternary, which is incorrectly using toggleInternal when toggle is set to false. Update to using nullish coallescing to check if toggle is defined rather than falsey.

[luukdv]

Thanks for the PR!

Could you maybe provide an example of a use case for this change?

[mnikolaus]

@luukdv This will fix trying to set toggle to false outside of Hamburger context. Example, user navigates to other page and you need to close the mobile menu and set hamburger to initial state (toggle: false).

[luukdv]

@mnikolaus That's already possible:

const [toggled, toggle] = useState();

<Hamburger toggled={toggled} toggle={toggle} />

// When user navigates to other page
toggle(false);

Can you provide a code example of the use case that's missing? Thanks!

[luukdv]

Closing due to inactivity.