fix: Fix Portal warning on Next.js

[diegohaz]

Closes #656

Work in progress

How to test?

TBD

Does this PR introduce breaking changes?

No

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit b628579:

Sandbox	Source
Reakit	Configuration
youthful-thunder-2qvsn	Issue #656

[ariakit-bot]

Deploy preview for reakit ready!

Built with commit b628579

https://deploy-preview-802--reakit.netlify.app

[github-actions]

Size Change: -15 B (0%)

Total Size: 256 kB

Filename	Size	Change
packages/reakit/dist/reakit.min.js	35.8 kB	-15 B (0%)

ℹ️ View Unchanged

Filename	Size	Change
packages/reakit-playground/dist/reakit-playground.min.js	187 kB	0 B
packages/reakit-system-bootstrap/dist/reakit-system-bootstrap.min.js	19.1 kB	0 B
packages/reakit-system-palette/dist/reakit-system-palette.min.js	8.85 kB	0 B
packages/reakit-system/dist/reakit-system.min.js	2.45 kB	0 B
packages/reakit-utils/dist/reakit-utils.min.js	3.39 kB	0 B

_{compressed-size-action}

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[Grsmto]

Is there anything we can do to help this one go through? 👀

[marcospassos]

@diegohaz any progress on this or any workaround?

[diegohaz]

@marcospassos I paused the work on this PR to focus on v2, which has a more robust fix for this issue. In the meantime, you can install the version created for this PR ("reakit": "https://pkg.csb.dev/reakit/reakit/commit/b6285795/reakit") and check if it's working properly.

I don't remember the error on the CI, but that's probably the only thing blocking this PR.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[espipj]

Hello! is there any timeline for this to be merged in the near future? I've tried this:

@marcospassos I paused the work on this PR to focus on v2, which has a more robust fix for this issue. In the meantime, you can install the version created for this PR ("reakit": "https://pkg.csb.dev/reakit/reakit/commit/b6285795/reakit") and check if it's working properly.

I don't remember the error on the CI, but that's probably the only thing blocking this PR.

and it wouldn't work. I might be able to give some help 😃

Edit: it does work! repro in codesandbox here

What's left on this PR? would be great if it could be merged on v1 so that #656 can be closed?

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.