Proposal: rename inline=false to portal=true

[giladgray]

What is "inline" really? This name has been confusing since the beginning, and we often talk about non-inlined Popovers, etc.

Implementationally, inline={false} means "render in a Portal to remove from DOM flow."

I therefore propose a new name for the prop: portal={true}. This makes the semantics obvious. Enabling portal will render the children inside a Portal; disabling will not use a Portal (preserving DOM flow). Boom done.

[adidahiya]

prefer usePortal, but generally sgtm

[sighrobot]

I cannot 👍 this enough – I completely missed the inline prop in the docs.

(A more DOM-accurate naming option could be renderInPortal)

[giladgray]

noting here to add callout on Portal docs about usePortal prop on other components.