WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@claviska
Copy link
Member

This PR fixes #1812 by reverting the component CSS files to Lit's preferred syntax. Since this updates the base class, I'll update pro after this is merged.

@claviska claviska requested a review from lindsaym-fa December 10, 2025 18:44
@claviska claviska self-assigned this Dec 10, 2025
@claviska claviska added the bug Something isn't working label Dec 10, 2025
@vercel
Copy link

vercel bot commented Dec 10, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
webawesome Ready Ready Preview Dec 11, 2025 0:56am

Copy link
Contributor

@lindsaym-fa lindsaym-fa left a comment

Choose a reason for hiding this comment

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

styles/utilities/size.css and styles/utilities/variants.css were both imported in native styles, so these are the results we're getting for native buttons and such now:

Image

I'd recommend:

  • Moving size.styles.ts and variants.styles.ts to the styles/component/ directory
  • Restoring size.css and variants.css in the styles/utilities/ directory and removing the :host() selector for each rule in these files

Does that make sense to you, @claviska? (And let me know if you'd like me to make those changes.)

@claviska
Copy link
Member Author

@lindsaym-fa nice catch. It looks like the visually hidden utilities were also affected so I updated those as well. Let me know if you see anything else!

@claviska
Copy link
Member Author

Note: I also removed the wa-utilities layer from the component CSS in d75fc9e.

Copy link
Contributor

@lindsaym-fa lindsaym-fa left a comment

Choose a reason for hiding this comment

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

One small comment. All else looks good!

Copy link
Contributor

@lindsaym-fa lindsaym-fa left a comment

Choose a reason for hiding this comment

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

@claviska claviska merged commit 895fb30 into next Dec 11, 2025
2 checks passed
@claviska claviska deleted the use-lit-styles branch December 11, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CSS Importing technique is defeating all stylesheet sharing

3 participants