Skip to content

Conversation

@castastrophe
Copy link
Collaborator

@castastrophe castastrophe commented Oct 27, 2025

Description

This update removes --mod-colorarea-* custom property hooks per SWC-1264, see also the RFC for extensible styling. Class selectors and variants remain unchanged; stories were refreshed to match the current API.

  • Remove all --mod-colorarea-* custom property hooks.
  • Reduces unnecessary re-mapping of custom properties in favor of using them directly

Breaking change: the --mod-colorarea-* override layer is removed. Consumers should set --spectrum-color-area-* variables directly where customization as needed.

Related issue(s)

  • fixes SWC-1264

Author's checklist

  • I have read the CONTRIBUTING and PULL_REQUESTS documents.
  • I have reviewed the Accessibility Practices for this feature, see: Aria Practices
  • I have added automated tests to cover my changes (or confirmed existing coverage is sufficient for styling-only changes).
  • I have included a well-written changeset if my change needs to be published.
  • I have included updated documentation if my change required it.

Reviewer's checklist

  • Includes a GitHub Issue with appropriate flag or Jira ticket number without a link
  • Includes thoughtfully written changeset if changes suggested include patch, minor, or major features
  • Automated tests cover all use cases and follow best practices for writing
  • Validated on all supported browsers
  • All VRTs are passing or approved

Validation steps

  • No visual or behavioral changes compared to the current spectrum-two baseline.

Regression testing

Validate:

  1. The documentation pages for at least two other components are still loading, including:

    • The pages render correctly, are accessible, and are responsive.
  2. If components have been modified, VRTs have been run on this branch:

    • VRTs have been run and looked at.
    • Any VRT changes have been accepted (by reviewer and/or PR author), or there are no changes.

@castastrophe castastrophe added run_vrt For use on PRs looking to kick off VRT ready-for-review S2 Spectrum 2 labels Oct 27, 2025
@changeset-bot
Copy link

changeset-bot bot commented Oct 27, 2025

⚠️ No Changeset found

Latest commit: de78ff4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes changesets to release 14 packages
Name Type
@spectrum-css/alertdialog Major
@spectrum-css/asset Major
@spectrum-css/assetcard Major
@spectrum-css/assetlist Major
@spectrum-css/avatar Major
@spectrum-css/badge Major
@spectrum-css/breadcrumb Major
@spectrum-css/colorarea Major
@spectrum-css/divider Major
@spectrum-css/miller Major
@spectrum-css/page Major
@spectrum-css/well Major
@spectrum-css/bundle Patch
@spectrum-css/preview Patch

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@castastrophe castastrophe force-pushed the castastrophe/feat-colorarea-modifier-removal branch from b2ed727 to d6e5a7f Compare October 27, 2025 17:02
@castastrophe castastrophe self-assigned this Oct 27, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

📚 Branch preview

PR #4298 has been deployed to Azure Blob Storage: https://spectrumcss.z13.web.core.windows.net/pr-4298/index.html.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

File metrics

Summary

Total size: 1.41 MB*
No change in file sizes

Package Size Minified Gzipped
colorarea 2.23 KB 2.08 KB 0.86 KB
colorhandle 3.92 KB 3.70 KB 1.04 KB
colorwheel 4.29 KB 3.99 KB 1.23 KB

File change details

colorarea

Filename Head Minified Gzipped Compared to base
index.css 2.23 KB 2.08 KB 0.86 KB 🟢 ⬇ 1.39 KB
metadata.json 0.82 KB - - 🟢 ⬇ 0.86 KB

colorhandle

Filename Head Minified Gzipped Compared to base
index.css 3.92 KB 3.70 KB 1.04 KB 🟢 ⬇ 0.18 KB
metadata.json 1.76 KB - - 🟢 ⬇ 0.11 KB

colorwheel

Filename Head Minified Gzipped Compared to base
index.css 4.29 KB 3.99 KB 1.23 KB 🟢 ⬇ 1.18 KB
metadata.json 1.37 KB - - 🟢 ⬇ 0.65 KB
* Size is the sum of all main files for packages in the library.
* An ASCII character in UTF-8 is 8 bits or 1 byte.

@castastrophe castastrophe force-pushed the castastrophe/feat-colorarea-modifier-removal branch 4 times, most recently from e7c0f9b to ccb055e Compare October 27, 2025 17:52
@castastrophe castastrophe force-pushed the castastrophe/feat-colorarea-modifier-removal branch 2 times, most recently from 01f57de to b9b3949 Compare October 27, 2025 18:49
@castastrophe castastrophe force-pushed the castastrophe/feat-colorarea-modifier-removal branch from 7158cba to de78ff4 Compare October 28, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review run_vrt For use on PRs looking to kick off VRT S2 Spectrum 2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants