Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug]: FluentUI popover does not work in Next@14.2.X, but it does work in next@14.1.X #32821

Closed
2 tasks done
Dejvovo opened this issue Sep 12, 2024 · 3 comments
Closed
2 tasks done

Comments

@Dejvovo
Copy link

Dejvovo commented Sep 12, 2024

Library

React Components / v9 (@fluentui/react-components)

System Info

System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh

Are you reporting an Accessibility issue?

None

Reproduction

https://stackblitz.com/edit/nextjs-bzkxez

Bug Description

Link to the code that reproduces this issue
https://stackblitz.com/edit/nextjs-bzkxez

To Reproduce
Link to stackblitz with version next 14.2:
https://stackblitz.com/edit/nextjs-bzkxez
Link to stackblitz with version next 14.1:
https://stackblitz.com/edit/nextjs-5igbub

Both projects are completely same. They are clean next.js projects with added fluentui/react-components.

In next version 14.1.X fluentui popover does work in version 14.2.X it doesnt.

Current vs. Expected behavior
Current behavior: Popover does not popoup as expected.

Expected behavior: Popover will popup.

Provide environment information
node: 18.20.3,
npm: 10.2.3

Also tried with newest node version
node: 22.2.0,
yarn: 4.2.2
Which area(s) are affected? (Select all that apply)
Not sure

Which stage(s) are affected? (Select all that apply)
Other (Deployed)

Additional context
Tried with yarn, pnpm, npm different versions. The only difference seems to be next version.
What I dont understand is that next app with fluentui must be used with dozens of people, yet i havent found any issue on this topic.

This is my first public issue, if I did something wrong Im ready for feedback.

Logs

No response

Requested priority

Normal

Products/sites affected

No response

Are you willing to submit a PR to fix?

no

Validations

  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • The provided reproduction is a minimal reproducible example of the bug.
@ewylie25
Copy link

I've noticed similar behavior and was just researching similar issues to decided where to report it... We're seeing the following:

  • linux containerized envs work for both production builds and nextjs dev mode
  • windows envs do not render the popover in nextjs dev node

When the popover does not render it appears that the fluent provider which contains the react portal is just missing from the DOM. The popover state indicates the popover is open so it's something about the react portals and surfaces not the event handlers.

Next 14.2 looks like it contains changes to the swc core packages which are OS dependent so I was thinking it had something to do with linux versus windows swc packages but sounds like you're seeing similar behavior on a linux OS so not sure.

I was planning to report this over here since I'd suspect it's more to do with this plugin than fluent itself - https://github.com/sopranopillow/fluentui-nextjs-appdir-plugin

@LYXOfficial
Copy link

I found the same bug, in Windows 10, Next.js 14.2 dev could not pop up the dialog window properly, when I downgraded to 14.1.0, the problem was solved, which delayed me for two days to debug.

@layershifter
Copy link
Member

Duplicate of #31930, see #31930 (comment).

@layershifter layershifter closed this as not planned Won't fix, can't repro, duplicate, stale Sep 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants