Skip to content

fix: issue #548#550

Merged
Yuyz0112 merged 1 commit intorrweb-io:masterfrom
YunFeng0817:iframe-patch
May 2, 2021
Merged

fix: issue #548#550
Yuyz0112 merged 1 commit intorrweb-io:masterfrom
YunFeng0817:iframe-patch

Conversation

@YunFeng0817
Copy link
Member

  1. Do not use virtual parent optimization if the mutation targets have iframe elements as children. This will cause some performance regression but will be easy to add and ship.
  2. If an iframe element has already been a child of a virtual parent, add the virtual parent back to the dom.

@Juice10 Please take a look at this PR.

1. Do not use virtual parent optimization if the mutation targets have iframe elements as children. This will cause some performance regression but will be easy to add and ship.
2. If an iframe element has already been a child of a virtual parent, add the virtual parent back to the dom.
Copy link
Member

@Juice10 Juice10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks great @Mark-Fenng! Love the pragmatic approach you and @Yuyz0112 came up with. We can always spend some time making it faster again, but at least now we'll have something that works.

@Yuyz0112 Yuyz0112 merged commit 7e46341 into rrweb-io:master May 2, 2021
@YunFeng0817 YunFeng0817 deleted the iframe-patch branch May 4, 2021 10:31
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.

3 participants