Skip to content

Merge named and shorthand class attributes#497

Closed
eboody wants to merge 2 commits into
lambda-fairy:mainfrom
eboody:fix-496-merge-class-attrs
Closed

Merge named and shorthand class attributes#497
eboody wants to merge 2 commits into
lambda-fairy:mainfrom
eboody:fix-496-merge-class-attrs

Conversation

@eboody
Copy link
Copy Markdown

@eboody eboody commented Apr 29, 2026

Summary

  • normalize named class attributes into the same rendering path as class shorthand so mixed forms emit one class attribute
  • preserve ordering across shorthand, literal, dynamic, and optional class fragments instead of emitting duplicate class attributes
  • add regression coverage for mixed shorthand with literal, dynamic, and optional named class values

Closes #496.

Testing

  • cargo test mixed_shorthand_and_
  • cargo test --test basic_syntax

eboody added 2 commits April 29, 2026 03:59
Normalize named class attributes into the same rendering path as class shorthand so mixed syntax produces a single stable class attribute.
Keep mixed shorthand and named class rendering on one normalization path while covering literal, dynamic, and optional named class values with regression tests.
@eboody eboody closed this by deleting the head repository May 5, 2026
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.

Mixed literal and dynamic classes emit duplicate class attributes

1 participant