Skip to content
This repository has been archived by the owner on Apr 8, 2024. It is now read-only.

core-dialog: Server rendered core dialog includes stringified onDialogToggle prop #261

Closed
esseb opened this issue May 28, 2019 · 2 comments
Assignees

Comments

@esseb
Copy link

esseb commented May 28, 2019

Example server rendered markup:

<core-dialog-react class="modal-dialog__dialog" onDialogToggle="function handleDialogClose() {
  // Some logic I've removed here
  props.onClose();
}" aria-label="React dialog">
 ...
</core-dialog-react>

This causes a React hydration warning because of a difference in my handleDialogClose function between server and client due to code minification etc.

Fails on master. Regressed between feb16c6 and 0f41aab.

@esseb esseb changed the title core-dialog: Server rendered core dialog included stringified onDialogToggle prop core-dialog: Server rendered core dialog includes stringified onDialogToggle prop May 28, 2019
@htor
Copy link
Contributor

htor commented May 28, 2019

thanks for this! looking into it..

@htor
Copy link
Contributor

htor commented Jun 3, 2019

closed by #265

@htor htor closed this as completed Jun 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants