Skip to content

fix(qwikloader): interactivity during streaming#8581

Open
Varixo wants to merge 1 commit intobuild/v2from
v2-fix-interactivity-during-streaming
Open

fix(qwikloader): interactivity during streaming#8581
Varixo wants to merge 1 commit intobuild/v2from
v2-fix-interactivity-during-streaming

Conversation

@Varixo
Copy link
Copy Markdown
Member

@Varixo Varixo commented Apr 27, 2026

No description provided.

@Varixo Varixo self-assigned this Apr 27, 2026
@Varixo Varixo added the V2 label Apr 27, 2026
@Varixo Varixo requested a review from a team as a code owner April 27, 2026 17:20
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 27, 2026

🦋 Changeset detected

Latest commit: 534e621

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

@maiieul maiieul moved this from Backlog to Waiting For Review in Qwik Development Apr 27, 2026
@Varixo Varixo linked an issue Apr 27, 2026 that may be closed by this pull request
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 27, 2026

Open in StackBlitz

@qwik.dev/core

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/core@8581

@qwik.dev/router

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/router@8581

eslint-plugin-qwik

npm i https://pkg.pr.new/QwikDev/qwik/eslint-plugin-qwik@8581

create-qwik

npm i https://pkg.pr.new/QwikDev/qwik/create-qwik@8581

@qwik.dev/optimizer

npm i https://pkg.pr.new/QwikDev/qwik/@qwik.dev/optimizer@8581

commit: 534e621

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 27, 2026

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

'@qwik.dev/core': patch
---

fix: correctly handle interactivty during html streaming
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

So before this change it only captured user events after the end of the stream?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

no, but it could resume qwik before the state streamed

};

const waitForContainerReady = (container: QContainerElement) =>
container.getAttribute('q:container') === 'paused' && doc.readyState === 'loading'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think it would be better to add code that calls container.ready() after the state. That way, a container can resume while the HTML is still streaming, like in multi containers

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Waiting For Review

Development

Successfully merging this pull request may close these issues.

[🐞] v2: qwikloader should wait until a container has state

3 participants