Skip to content

Conversation

castastrophe
Copy link
Collaborator

@castastrophe castastrophe commented Oct 16, 2025

Description

This update removes --mod-actiongroup-* 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-actiongroup-* custom property hooks.
  • Reduces unnecessary re-mapping of custom properties in favor of using them directly

Breaking change: the --mod-actiongroup-* override layer is removed. Consumers should set --spectrum-actiongroup-* 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 self-assigned this Oct 16, 2025
@castastrophe castastrophe added size-2 S ~6-18hrs; not hard or time consuming, one or two work days to complete. run_vrt For use on PRs looking to kick off VRT ready-for-review S2 Spectrum 2 labels Oct 16, 2025
Copy link

changeset-bot bot commented Oct 16, 2025

⚠️ No Changeset found

Latest commit: 08ad101

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 no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

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

Copy link
Contributor

github-actions bot commented Oct 16, 2025

📚 Branch preview

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

@castastrophe castastrophe force-pushed the castastrophe/feat-actiongroup-modifier-removal branch 2 times, most recently from 20faae7 to 741f6f2 Compare October 16, 2025 17:01
Copy link
Contributor

github-actions bot commented Oct 16, 2025

File metrics

Summary

Total size: 1.43 MB*

Package Size Minified Gzipped
actiongroup 4.79 KB 4.55 KB 0.92 KB

actiongroup

Filename Head Minified Gzipped Compared to base
index.css 4.79 KB 4.55 KB 0.92 KB 🟢 ⬇ 2.88 KB
metadata.json 2.39 KB - - 🟢 ⬇ 0.90 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-actiongroup-modifier-removal branch from 741f6f2 to 08ad101 Compare October 16, 2025 17:07
@castastrophe castastrophe marked this pull request as ready for review October 16, 2025 17:10
@castastrophe castastrophe merged commit 6203506 into spectrum-two Oct 16, 2025
15 checks passed
@castastrophe castastrophe deleted the castastrophe/feat-actiongroup-modifier-removal branch October 16, 2025 17:20
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 size-2 S ~6-18hrs; not hard or time consuming, one or two work days to complete.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants