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

[Dialog]support ssr please #4205

Closed
smat0096 opened this issue Dec 5, 2022 · 1 comment · Fixed by #4923
Closed

[Dialog]support ssr please #4205

smat0096 opened this issue Dec 5, 2022 · 1 comment · Fixed by #4923
Assignees
Labels
💡 Feature Request New feature or request Next release issues to close in next release

Comments

@smat0096
Copy link

smat0096 commented Dec 5, 2022

Component

Dialog

Steps to reproduce

document is undefined in ssr env, at overlay.js used such code:
useListener(document, 'mousedown', clickEvent, false, !!(visible && overlayRef.current && (canCloseByOutSideClick || hasMask && canCloseByMask)));

@lakerswgq lakerswgq added the 💡 Feature Request New feature or request label Dec 5, 2022
@eternalsky eternalsky added the Next release issues to close in next release label Jul 29, 2024
@FairyYang FairyYang self-assigned this Aug 19, 2024
FairyYang added a commit to FairyYang/next that referenced this issue Aug 30, 2024
@eternalsky eternalsky linked a pull request Aug 30, 2024 that will close this issue
FairyYang added a commit to FairyYang/next that referenced this issue Sep 3, 2024
FairyYang added a commit to FairyYang/next that referenced this issue Sep 4, 2024
FairyYang added a commit to FairyYang/next that referenced this issue Sep 5, 2024
FairyYang added a commit to FairyYang/next that referenced this issue Sep 5, 2024
@eternalsky
Copy link
Contributor

the code above is written in didMount lifeCycle, and will not affect ssr. but we review the whole code of Dialog, and fix all problems which will result in error in ssr.

luolin-ck pushed a commit to luolin-ck/next that referenced this issue Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💡 Feature Request New feature or request Next release issues to close in next release
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

4 participants