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: rendering attribute value from array of classes from spread props #7310

Merged
merged 1 commit into from
Feb 5, 2025

Conversation

Varixo
Copy link
Member

@Varixo Varixo commented Feb 4, 2025

No description provided.

@Varixo Varixo self-assigned this Feb 4, 2025
@Varixo Varixo requested a review from a team as a code owner February 4, 2025 20:28
Copy link

changeset-bot bot commented Feb 4, 2025

🦋 Changeset detected

Latest commit: 7c241ad

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

This PR includes changesets to release 5 packages
Name Type
@qwik.dev/core Patch
eslint-plugin-qwik Patch
@qwik.dev/react Patch
@qwik.dev/router Patch
create-qwik Patch

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 marked this pull request as draft February 4, 2025 21:22
@Varixo Varixo force-pushed the v2-class-array-rest-props branch 3 times, most recently from 3573ed4 to 0f4fbce Compare February 5, 2025 16:08
Copy link

pkg-pr-new bot commented Feb 5, 2025

Open in Stackblitz

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

commit: 7c241ad

Copy link
Contributor

github-actions bot commented Feb 5, 2025

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

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

@Varixo Varixo marked this pull request as ready for review February 5, 2025 16:24
@@ -48,6 +49,10 @@ export abstract class _SharedContainer implements Container {
return trackSignalAndAssignHost(signal, subscriber, property, this, data);
}

serializeAttributeValue(key: string, value: any, styleScopedId?: string | null) {
Copy link
Member

Choose a reason for hiding this comment

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

Why not simply re-export?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, I forgot about reverting that change

@Varixo Varixo force-pushed the v2-class-array-rest-props branch from 0f4fbce to e6d5b10 Compare February 5, 2025 18:42
@Varixo Varixo force-pushed the v2-class-array-rest-props branch from e6d5b10 to 7c241ad Compare February 5, 2025 18:47
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.

LGTM

@wmertens wmertens merged commit 0d6056e into build/v2 Feb 5, 2025
19 checks passed
@wmertens wmertens deleted the v2-class-array-rest-props branch February 5, 2025 19:31
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.

2 participants