fix: 解决 React 等将事件处理统一放在 Document 上, 导致 shadowDom 下事件不触发的问题 #35
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
我一直以为 react 最新版本已经不再将时间收集到 document 上了，可否提供一个最简用例，先确认下最新版本是否有问题 |
|
我合并啦，阿里嘎多，预想中会有更多的坑，如果作为 polyfill 就好了 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
主要是解决这个问题：facebook/react#9242
在 ShadowDom 下渲染 React 应用的时候，会发现 React 的所有事件都无法触发。
这个是由于 ShadowDom 里触发的事件，在外层拿到 event.target 的时候，只会拿到 host，所以导致了 react 的事件调度出现了问题。