Skip to content

Conversation

castastrophe
Copy link
Collaborator

@castastrophe castastrophe commented Oct 22, 2025

Description

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

Breaking change: the --mod-page-* override layer is removed. Consumers should set --spectrum-page-* 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 22, 2025
@changeset-bot
Copy link

changeset-bot bot commented Oct 22, 2025

⚠️ No Changeset found

Latest commit: 06e9af0

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 11 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/miller Major
@spectrum-css/well Major
@spectrum-css/page 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

@github-actions
Copy link
Contributor

📚 Branch preview

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

@github-actions
Copy link
Contributor

File metrics

Summary

Total size: 1.41 MB*

Package Size Minified Gzipped
page 0.74 KB 0.73 KB 0.47 KB

page

Filename Head Minified Gzipped Compared to base
index.css 0.74 KB 0.73 KB 0.47 KB 🟢 ⬇ 0.23 KB
metadata.json 0.21 KB - - 🟢 ⬇ 0.15 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 merged commit bdc66c9 into spectrum-two Oct 22, 2025
17 of 26 checks passed
@castastrophe castastrophe deleted the castastrophe/feat-page-modifier-removal branch October 22, 2025 20:53
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.

1 participant