Better support for using with exactOptionalPropertyTypes enabled

[hornta]

We recently enabled exactOptionalPropertyTypes for our code base because we had a bug that could have been prevented if that flag was enabled in our tsconfig.json. We've fixed some of the issues caused by enabling this flag now and now we found it caused a problem with when we pass undefined as a property to <FloatingFocusManager>.

We have this code in our codebase and this is a issue because the type for initialFocus is doesn't contain an explicit undefined.

<FloatingFocusManager
  context={floatingContext}
  initialFocus={
    typeof children === "function" ? initialElementRef : undefined
  }
>

We can get around this problem by moving some code around like below but it's not pretty.

const inner = ...

if(typeof children === "function") {
  return <FloatingFocusManager context={floatingContext} initialFocus={initialElementRef}>{inner}</FloatingFocusManager>
} else {
  return <FloatingFocusManager context={floatingContext}>{inner}</FloatingFocusManager>
}

I can provide a PR to make it easier for code bases using exactOptionalPropertyTypes to use floating-ui.

[atomiks]

So this only involves adding | undefined to the types instead of relying on ?: alone?

[hornta]

More or less, yes!