Skip to content

Conversation

Varixo
Copy link
Member

@Varixo Varixo commented Oct 4, 2025

Rewrite the two-pointer algorithm for setting and removing attributes on vnodes

@Varixo Varixo self-assigned this Oct 4, 2025
@Varixo Varixo requested a review from a team as a code owner October 4, 2025 14:03
Copy link

changeset-bot bot commented Oct 4, 2025

🦋 Changeset detected

Latest commit: 4794f2a

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

Copy link
Contributor

github-actions bot commented Oct 4, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

Copy link

pkg-pr-new bot commented Oct 4, 2025

Open in StackBlitz

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

commit: 4794f2a

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.

It looks amazing 🤩
Thanks @Varixo

Copy link
Member

@wmertens wmertens left a comment

Choose a reason for hiding this comment

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

really easy to follow the algo with the comments, great!

LGTM

@wmertens wmertens merged commit 8f60e5d into build/v2 Oct 5, 2025
49 of 55 checks passed
@wmertens wmertens deleted the v2-fix-attributes-edge-cases branch October 5, 2025 15:19
@github-project-automation github-project-automation bot moved this from Waiting For Review to Done in Qwik Development Oct 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[🐞] [v2] Error raised when a prop is used to conditionally select class, only when there are more than 6 classes on both sides of condition.
3 participants