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

preserve adopted styles from nodes being removed when virtual dom is in use #36

Merged
merged 2 commits into from
Feb 3, 2025

Conversation

megboehlert
Copy link

No description provided.

@pendo-ops
Copy link

Failed pendo.io/jira-ref check. Please correct Jira references in the following commits:

f8213ea

1 similar comment
@pendo-ops
Copy link

Failed pendo.io/jira-ref check. Please correct Jira references in the following commits:

f8213ea

@pendo-ops
Copy link

Failed pendo.io/jira-ref check. Please correct Jira references in the following commits:

f8213ea
08be4ad

@guntherjh
Copy link

Talked f2f. My only potential concern here would be performance, but given the timing of the fix with everything else we have going on, I'm good to defer verifying that for now. Since we'll be releasing this as part of the beta player, this change is only being roled out to a very small subset of users, and we have the option to take them out of the beta segment if anything catastrophic happens.

@megboehlert megboehlert force-pushed the mb-preserve-adopted-styles branch from 08be4ad to 247736a Compare February 3, 2025 15:21
@pendo-ops
Copy link

Failed pendo.io/jira-ref check. Please correct Jira references in the following commits:

247736a

@pendo-ops
Copy link

Failed pendo.io/jira-ref check. Please correct Jira references in the following commits:

247736a
9fb6f48

@guntherjh
Copy link

[merge]

@pendo-ops pendo-ops merged commit 01a643b into pendo-main Feb 3, 2025
1 of 2 checks passed
@pendo-ops
Copy link

Branch mb-preserve-adopted-styles deleted

@pendo-ops pendo-ops deleted the mb-preserve-adopted-styles branch February 3, 2025 15:30
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.

4 participants