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 for attachShadow no longer adding childNodes to shadowRoot #211

Merged
merged 3 commits into from
Jul 17, 2023

Conversation

ttraenkler
Copy link
Contributor

@ttraenkler ttraenkler commented Jun 25, 2023

fix for #178: no longer adds the childNodes of the element to its shadowRoot when attaching a new shadowRoot

@ttraenkler ttraenkler changed the title fix for issue #178 attachShadow adding childNodes to shadowRoot fix for attachShadow no longer adding childNodes to shadowRoot Jun 25, 2023
@luwes
Copy link
Contributor

luwes commented Jul 16, 2023

fixed this bug in my branch as well
luwes@98f94da

Copy link
Contributor

@luwes luwes left a comment

Choose a reason for hiding this comment

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

LGTM!

@ttraenkler
Copy link
Contributor Author

ttraenkler commented Jul 16, 2023

@luwes Thanks! I see you also maintain work on improving shadow DOM support. Are you interested in collaborating on this? I am currently working on supporting element.getInnerHTML() #210 and improved shadowRoot support here https://github.com/loopdive/linkedom which could be merged upstream once it's complete.

@WebReflection WebReflection merged commit 2be5c65 into WebReflection:main Jul 17, 2023
@WebReflection
Copy link
Owner

it's up and running

@luwes
Copy link
Contributor

luwes commented Jul 17, 2023

thanks @WebReflection!


@ttraenkler happy to collaborate on that.

The branch with most of my fixes and improvements are here https://github.com/luwes/linkedom/commits/wesc

It's for a project called WeSC https://github.com/luwes/wesc. One of the first goals is to add DSD easily in any project.

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