[Select] Add non modal mode in popper position

[joaom00]

Description

As suggested here #2004 (comment), this PR adds non-modal mode only for the popper position since that mode for item-aligned position needs to be thought a little more.

Closes #1927

[joaom00]

I have seen a problem on mobile in non-modal mode where the Trigger gets focus after being opened. Maybe I should put this line onFocusOutside={(event) => event.preventDefault()} in both modal and non-modal mode?

I noticed that this same issue also occurs in the dropdown menu, I believe it is because of react 17? In the documentation it doesn't seem to have any issues 🤔

I think this is an issue #2241

[kachkaev]

👋 @benoitgrelard! Any chance this PR could be looked into? It potentially fixes a rather serious performance issue – see #1634 (comment) and below for details. I use Radix Select in a project via https://ui.shadcn.com/docs/components/select. When a page has heavy markup like a large table, clicking on select makes the app freeze even on a pretty recent MacBookPro with M1. The lag is especially visible in Safari, but it’s also noticeable on Chrome and FF (especially with x4 throttling).

[frykten]

@joaom00 Olá! This is just a ping 🙂 is this PR/subject still alive?

[joaom00]

@frykten Yes definitely. Unfortunately, I think the team didn't have time to review the PR...

[SaveliiLukash]

Came here from #2241

Thanks for the hint on onFocusOutside={ (event) => event.preventDefault() }. Might be a temporary, yet hacky solution, which is far from ideal.

Looking forward for this issue to be handled by the library itself.

[maschad96]

beep boop bump, this is a nice contribution – any chance it could get a review soon @benoitgrelard @andy-hook?

currently resorting to using DropdownMenu for the non-modal support when semantically I should be using Select

[andreareginato]

Any chance for this to be merged?

[CodingAleCR]

I would also prefer it if this fix was checked and worked/merged. Is there anyway we can help to push it over the line?

[bmutahhar]

I am facing the same issue. Any updates on when this will be merged?

[atancredi]

Hi all! This is just a ping, any updates on a possible merge?

[donghoon-song]

Any..

[ttpss930141011]

Will be appreciated if the PR is merged

[Taaqif]

Any updates on when this might be merged?

[chaance]

This one is on my radar, though I'd prefer to resolve it without additional wrapper components. One of our goals over the coming months is to reduce React tree bloat to improve the React dev tools experience.