Skip to content

Conversation

@Varixo
Copy link
Member

@Varixo Varixo commented Sep 10, 2025

Pass first child of shadow root's as container. In other case the tree walker returns the same container element for nextNode function.

@changeset-bot
Copy link

changeset-bot bot commented Sep 10, 2025

🦋 Changeset detected

Latest commit: bedec7c

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@Varixo Varixo moved this from Backlog to Waiting For Review in Qwik Development Sep 10, 2025
// Try to get Qwik container if available
try {
const container = _getDomContainer(doc.documentElement);
const container = _getDomContainer(doc.body.firstElementChild!);
Copy link
Member Author

Choose a reason for hiding this comment

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

its just docs parser fix, not related

Copy link
Member

@gioboa gioboa left a comment

Choose a reason for hiding this comment

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

Looking forward to trying this fix 👏💪
You rock @Varixo

@pkg-pr-new
Copy link

pkg-pr-new bot commented Sep 10, 2025

Open in StackBlitz

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/core@7937
npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/router@7937
npm i https://pkg.pr.new/QwikDev/qwik/eslint-plugin-qwik@7937
npm i https://pkg.pr.new/QwikDev/qwik/create-qwik@7937

commit: bedec7c

@github-actions
Copy link
Contributor

github-actions bot commented Sep 10, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name Status Preview Last Commit
qwik-docs ✅ Ready (View Log) Visit Preview bedec7c

@Varixo Varixo merged commit b8b568e into build/v2 Sep 10, 2025
20 checks passed
@Varixo Varixo deleted the v2-resuming-shadow-root branch September 10, 2025 18:58
@github-project-automation github-project-automation bot moved this from Waiting For Review to Done in Qwik Development Sep 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants