- 
                Notifications
    
You must be signed in to change notification settings  - Fork 207
 
feat(icon): remove modifiers from the API [SWC-1264] #4288
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: spectrum-two
Are you sure you want to change the base?
Conversation
          
 | 
    
| 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/divider | Major | 
| @spectrum-css/icon | 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
          📚 Branch previewPR #4288 has been deployed to Azure Blob Storage: https://spectrumcss.z13.web.core.windows.net/pr-4288/index.html.  | 
    
c7d0b47    to
    52241c4      
    Compare
  
    
          File metricsSummaryTotal size: 1.41 MB* 
 File change detailsalertbanner
 icon
 pickerbutton
 table
 tag
 * An ASCII character in UTF-8 is 8 bits or 1 byte.  | 
    
9b42ed8    to
    2b3ff45      
    Compare
  
    eb322f4    to
    945da3b      
    Compare
  
    86bc092    to
    71a231f      
    Compare
  
    39723a2    to
    841dca9      
    Compare
  
    14740cb    to
    857bd63      
    Compare
  
    ebf7a32    to
    20888f4      
    Compare
  
    20888f4    to
    de39bef      
    Compare
  
    
Description
This update removes
--mod-icon-*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.--mod-icon-*custom property hooks.Breaking change: the
--mod-icon-*override layer is removed. Consumers should set--spectrum-icon-*variables directly where customization as needed.Related issue(s)
Author's checklist
Reviewer's checklist
patch,minor, ormajorfeaturesValidation steps
spectrum-twobaseline.Regression testing
Validate:
The documentation pages for at least two other components are still loading, including:
If components have been modified, VRTs have been run on this branch: