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

fix(react): remove the cleanup of ShadowSVG's effect #162

Merged
merged 1 commit into from
Dec 8, 2021

Conversation

zjffun
Copy link
Contributor

@zjffun zjffun commented Dec 8, 2021

我在将 designable 用在 umi 项目中发现会报错 Uncaught DOMException: Failed to execute 'attachShadow' on 'Element': Shadow root cannot be created on a host which already hosts a shadow tree.,调查了一下发现在已经 attachShadow 的元素上再次调用 attachShadow 会报错这个错误。

我试了一下 普通情况 无法复现这个问题,因为 ref.current 总是 null.

  • Ensure the pull request title and commit message follow the Commit Specific in English.
  • Fork the repo and create your branch from main.
  • If you've added code that should be tested, add tests!
  • If you've changed APIs, update the documentation.
  • Ensure the test suite passes (npm test).
  • Make sure your code lints (npm run lint) - we've done our best to make sure these rules match our internal linting guidelines.

@netlify
Copy link

netlify bot commented Dec 8, 2021

✔️ Deploy Preview for designable-fusion ready!

🔨 Explore the source changes: 7d77533

🔍 Inspect the deploy log: https://app.netlify.com/sites/designable-fusion/deploys/61b0595a4c2188000700a63f

😎 Browse the preview: https://deploy-preview-162--designable-fusion.netlify.app/

@netlify
Copy link

netlify bot commented Dec 8, 2021

✔️ Deploy Preview for designable-antd ready!

🔨 Explore the source changes: 7d77533

🔍 Inspect the deploy log: https://app.netlify.com/sites/designable-antd/deploys/61b0595a956ea700074311b2

😎 Browse the preview: https://deploy-preview-162--designable-antd.netlify.app/

@janryWang janryWang merged commit b982a2e into alibaba:main Dec 8, 2021
DivXPro added a commit to toy-box/designable that referenced this pull request Dec 22, 2021
* main:
  chore(versions): 😊 publish v1.0.0-beta.42
  feat: support setNpmCDNRegistry
  chore(versions): 😊 publish v1.0.0-beta.41
  fix: fix isBehavior
  fix(formily): set default language (alibaba#163)
  fix(react): remove the cleanup of ShadowSVG's effect (alibaba#162)
  chore(versions): 😊 publish v1.0.0-beta.40
  chore: add sandbox example
  fix(react): fix NodePathWidget style
  fix(react): fix root style
  fix(react): fix prettier cdn version
  chore(versions): 😊 publish v1.0.0-beta.39
  fix(react): create attach-shadow repeatedly (alibaba#161)
  chore(versions): 😊 publish v1.0.0-beta.38
  fix(react): fix position props in StudioPanel (alibaba#155)
  chore: improve node reisze
  feat: support resizable to designerProps
  fix(react): fix studiopanel className setting (alibaba#153)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants