Skip to content
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

refactor(ui-icons): s2 assets sourced from repo #2642

Closed
wants to merge 42 commits into from

Conversation

castastrophe
Copy link
Collaborator

Description

How and where has this been tested?

Please tag yourself on the tests you've marked complete to confirm the tests have been run by someone other than the author.

Validation steps

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.
  1. 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.

Screenshots

To-do list

  • I have read the contribution guidelines.
  • I have updated relevant storybook stories and templates.
  • I have tested these changes in Windows High Contrast mode.
  • If my change impacts other components, I have tested to make sure they don't break.
  • If my change impacts documentation, I have updated the documentation accordingly.
  • ✨ This pull request is ready to merge. ✨

@castastrophe castastrophe force-pushed the refactor-ui-icons branch 4 times, most recently from 1ccd869 to 93697c7 Compare April 5, 2024 20:29
@castastrophe castastrophe changed the title refactor(ui-icons): s2 migration of assets refactor(ui-icons): s2 assets sourced from repo Apr 11, 2024
@castastrophe castastrophe force-pushed the refactor-ui-icons branch 2 times, most recently from 031beb2 to 4e9b88c Compare April 11, 2024 19:14
@castastrophe castastrophe added the S2 Spectrum 2 label Apr 12, 2024
@pfulton pfulton force-pushed the spectrum-two branch 3 times, most recently from f3dd9ff to 9931a8e Compare April 19, 2024 18:22
@castastrophe castastrophe force-pushed the spectrum-two branch 4 times, most recently from 8f768fd to 8768360 Compare April 26, 2024 18:31
pfulton and others added 12 commits April 26, 2024 16:37
Includes new color values for Spectrum 2
BREAKING CHANGE: migrates `Action Group` to Spectrum 2, Removes some mod properties
BREAKING CHANGE: migrates Button Group to Spectrum 2

Also:
* docs(buttongroup): expand chromatic coverage
* refactor(buttongroup): remove extra css classes
BREAKING CHANGE: uses Spectrum 2 tokens
 - @spectrum-css/preview@7.4.2-next.0
 - @spectrum-css/actiongroup@5.0.0-next.0
 - @spectrum-css/buttongroup@7.0.0-next.0
 - @spectrum-css/tokens@14.0.0-next.0
 - @spectrum-css/bundle-builder@6.3.1-next.0
 - @spectrum-css/component-builder-simple@5.0.1-next.0
 - @spectrum-css/component-builder@6.0.1-next.0
BREAKING CHANGE: migrates Close Button to Spectrum 2

Additionally:
* test: increase chromatic coverage
* fix(closebutton): pass staticColor as arg for SB display
* chore(closebutton): remove themes dir
* docs(closebutton): adds s2 migration notes
* chore(closebutton): specify s2 tokens release for dependency

---------

Co-authored-by: Patrick Fulton <pfulton@adobe.com>
* chore: migrate gray-50 to gray-25

Migrates any instance of `--spectrum-gray-50` to use
`--spectrum-gray-25` as per the S2 migration guide

* chore: migrate gray-75 to gray-50

Migrates usages of `--spectrum-gray-75` to use
`--spectrum-gray-50` as per the s2 migration guide.

* chore: migrate gray-100 to gray-75

Migrates usages of `--spectrum-gray-100` to use
`--spectrum-gray-75` as per the s2 migration guide

* chore: migrate gray-200 to gray-100

Migrates usages of `--spectrum-gray-200` to use
`--spectrum-gray-100` as per the s2 migration guide

* chore: migrate gray-300 to gray-200

Migrates usages of `--spectrum-gray-300` to use
`spectrum-gray-200` as per the s2 migration guide

* chore(infieldbutton): gray-300 to gray-200
pfulton and others added 20 commits April 26, 2024 16:37
* feat(tokens): use spectrum tokens beta 21
* chore(tokens): add custom token for corner rounding
 - @spectrum-css/preview@8.1.0-next.0
 - @spectrum-css/accordion@4.2.7-next.1
 - @spectrum-css/actionbar@7.2.5-next.1
 - @spectrum-css/actionbutton@5.2.7-next.1
 - @spectrum-css/actiongroup@5.0.0-next.2
 - @spectrum-css/actionmenu@5.1.4-next.1
 - @spectrum-css/alertbanner@1.1.42-next.1
 - @spectrum-css/alertdialog@1.2.5-next.1
 - @spectrum-css/asset@4.0.1-next.1
 - @spectrum-css/assetcard@3.1.5-next.1
 - @spectrum-css/assetlist@5.2.5-next.1
 - @spectrum-css/avatar@6.1.6-next.1
 - @spectrum-css/badge@3.2.6-next.1
 - @spectrum-css/breadcrumb@8.2.6-next.1
 - @spectrum-css/button@14.0.0-next.3
 - @spectrum-css/buttongroup@7.0.0-next.2
 - @spectrum-css/calendar@4.2.6-next.1
 - @spectrum-css/card@7.0.3-next.1
 - @spectrum-css/checkbox@14.0.0-next.3
 - @spectrum-css/clearbutton@5.1.5-next.1
 - @spectrum-css/closebutton@5.0.0-next.1
 - @spectrum-css/coachindicator@1.1.6-next.1
 - @spectrum-css/coachmark@6.1.6-next.1
 - @spectrum-css/colorarea@4.1.6-next.1
 - @spectrum-css/colorhandle@7.1.5-next.1
 - @spectrum-css/colorloupe@4.2.5-next.1
 - @spectrum-css/colorslider@5.1.6-next.1
 - @spectrum-css/colorwheel@3.1.6-next.1
 - @spectrum-css/combobox@2.1.7-next.1
 - @spectrum-css/commons@9.1.4-next.0
 - @spectrum-css/contextualhelp@2.1.6-next.1
 - @spectrum-css/datepicker@2.1.5-next.1
 - @spectrum-css/dial@2.2.5-next.1
 - @spectrum-css/dialog@9.2.5-next.1
 - @spectrum-css/divider@2.2.6-next.1
 - @spectrum-css/dropindicator@4.1.5-next.1
 - @spectrum-css/dropzone@5.2.6-next.1
 - @spectrum-css/fieldgroup@4.2.5-next.1
 - @spectrum-css/fieldlabel@8.0.0-next.1
 - @spectrum-css/floatingactionbutton@1.2.6-next.1
 - @spectrum-css/helptext@4.1.6-next.1
 - @spectrum-css/icon@6.0.6-next.1
 - @spectrum-css/illustratedmessage@6.1.6-next.1
 - @spectrum-css/infieldbutton@4.2.5-next.1
 - @spectrum-css/inlinealert@7.1.7-next.1
 - @spectrum-css/link@4.2.6-next.1
 - @spectrum-css/logicbutton@3.3.5-next.1
 - @spectrum-css/menu@6.1.6-next.1
 - @spectrum-css/miller@5.1.5-next.1
 - @spectrum-css/modal@4.2.7-next.1
 - @spectrum-css/opacitycheckerboard@1.1.6-next.1
 - @spectrum-css/page@7.1.5-next.1
 - @spectrum-css/pagination@7.1.7-next.1
 - @spectrum-css/picker@7.2.8-next.1
 - @spectrum-css/pickerbutton@4.1.6-next.1
 - @spectrum-css/popover@6.2.6-next.1
 - @spectrum-css/progressbar@3.1.6-next.1
 - @spectrum-css/progresscircle@2.2.4-next.1
 - @spectrum-css/radio@8.1.6-next.1
 - @spectrum-css/rating@4.2.5-next.1
 - @spectrum-css/search@6.2.5-next.1
 - @spectrum-css/sidenav@4.2.5-next.1
 - @spectrum-css/site@4.2.5-next.1
 - @spectrum-css/slider@4.3.6-next.1
 - @spectrum-css/splitview@4.2.5-next.1
 - @spectrum-css/statuslight@6.1.7-next.1
 - @spectrum-css/steplist@4.1.5-next.1
 - @spectrum-css/stepper@5.1.6-next.1
 - @spectrum-css/swatch@5.1.6-next.1
 - @spectrum-css/swatchgroup@2.1.6-next.1
 - @spectrum-css/switch@4.2.6-next.1
 - @spectrum-css/table@5.2.6-next.1
 - @spectrum-css/tabs@4.1.5-next.1
 - @spectrum-css/tag@8.1.6-next.1
 - @spectrum-css/taggroup@4.1.6-next.1
 - @spectrum-css/textfield@6.1.7-next.1
 - @spectrum-css/thumbnail@5.2.5-next.1
 - @spectrum-css/toast@9.1.26-next.1
 - @spectrum-css/tooltip@5.3.6-next.1
 - @spectrum-css/tray@2.2.8-next.1
 - @spectrum-css/treeview@9.2.7-next.1
 - @spectrum-css/typography@5.1.6-next.1
 - @spectrum-css/underlay@3.2.5-next.1
 - @spectrum-css/well@4.1.5-next.1
 - @spectrum-css/tokens@14.0.0-next.3
 - @spectrum-css/bundle-builder@7.0.3-next.1
 - @spectrum-css/component-builder-simple@5.0.6-next.1
 - @spectrum-css/component-builder@7.0.3-next.1
…2559)

* feat(cornerrounding): documentation, custom tokens

- feat(cornerrounding): add custom token
- chore(cornerrounding): adds dist
- docs(cornerrounding): add file strucutre for foundations
- feat(cornerrounding): override token value
  - Sets custom var to override token spectrum-corner-radius-full
- chore(cornerrounding): add override to dist
  - Adds custom var to override token value to dist
- chore(cornerrounding): adjust file structure
  - Also renames file
- feat(cornerrounding): add Foundations story for corner-rounding
- feat(cornerrounding): add action buttons and checkboxes
  - Adds action buttons and checkboxes stories to demonstrate corner rounding
- docs(cornerrounding): update corner-rounding docs
  - better utilize storybook features
  - clean up checkbox and action button stories
  - hide stories in corner-rounding directory
  - improve table styling and css token use
  - improve corner rounding documentation
  - add tables for component size tokens
- fix(cornerrounding): remove mdx for tables
- fix(cornerrounding): use sentence case
- feat(cornerrounding): use custom token on close button
- docs: add alias note
- docs: remove reference to non-alias tokens
- feat(actionbutton): partial migration for corner rounding only
- feat(checkbox): partial migration for corner rounding only
- chore(closebutton): update custom var post-refactor
- fix: token rebase issue
- chore(actionbutton,checkbox): update package version
- chore: use corner-radius alias token
- chore: update tokens version
- chore: revert unneeded component version change
- docs: design requested updates, show token with example
- docs: move tables to stories, inline spacing for tables

* fix: includes error

---------

Co-authored-by: Melissa Thompson <mthompson@heysparkbox.com>
Remove dates from in front of version numbers on migration guides, as
we're adjusting to this format going forward. Includes removal of the
placeholder "x/x/2024" dates that were added. The button component
was excluded from this update as they're already removed in the button
s2 migration PR.
* feat(opacitycheckerboard): small t-shirt size squares
* docs(opacitycheckerboard): chromatic coverage for t-shirt size
* feat: add example gradients for static black and white

Add new gradient backgrounds, displayed for static black and static
white variants. These are used for examples only. This adds CSS custom
properties available globally within Storybook and sets them on the
existing decorator.

* docs(fieldlabel): support static colors decorator in storybook

Change Field label stories that show static black and static white,
so they work with the recently added decorator that changes the main
Storybook background.

* docs(button): adjust static colors template

Adjust static colors template to better handle the static color
decorator and gradients.

* chore(fieldlabel): apply eslint indentation changes to stories
 - @spectrum-css/preview@8.1.0-next.1
 - @spectrum-css/accordion@4.2.7-next.2
 - @spectrum-css/actionbar@7.2.5-next.2
 - @spectrum-css/actionbutton@5.2.7-next.2
 - @spectrum-css/actiongroup@5.0.0-next.3
 - @spectrum-css/actionmenu@5.1.4-next.2
 - @spectrum-css/alertbanner@1.1.42-next.2
 - @spectrum-css/alertdialog@1.2.5-next.2
 - @spectrum-css/asset@4.0.1-next.2
 - @spectrum-css/assetcard@3.1.5-next.2
 - @spectrum-css/assetlist@5.2.5-next.2
 - @spectrum-css/avatar@6.1.6-next.2
 - @spectrum-css/badge@3.2.6-next.2
 - @spectrum-css/breadcrumb@8.2.6-next.2
 - @spectrum-css/button@14.0.0-next.4
 - @spectrum-css/buttongroup@7.0.0-next.3
 - @spectrum-css/calendar@4.2.6-next.2
 - @spectrum-css/card@7.0.3-next.2
 - @spectrum-css/checkbox@14.0.0-next.4
 - @spectrum-css/clearbutton@5.1.5-next.2
 - @spectrum-css/closebutton@5.0.0-next.2
 - @spectrum-css/coachindicator@1.1.6-next.2
 - @spectrum-css/coachmark@6.1.6-next.2
 - @spectrum-css/colorarea@4.1.6-next.2
 - @spectrum-css/colorhandle@7.1.5-next.2
 - @spectrum-css/colorloupe@4.2.5-next.2
 - @spectrum-css/colorslider@5.1.6-next.2
 - @spectrum-css/colorwheel@3.1.6-next.2
 - @spectrum-css/combobox@2.1.7-next.2
 - @spectrum-css/commons@9.1.4-next.1
 - @spectrum-css/contextualhelp@2.1.6-next.2
 - @spectrum-css/datepicker@2.1.5-next.2
 - @spectrum-css/dial@2.2.5-next.2
 - @spectrum-css/dialog@9.2.5-next.2
 - @spectrum-css/divider@2.2.6-next.2
 - @spectrum-css/dropindicator@4.1.5-next.2
 - @spectrum-css/dropzone@5.2.6-next.2
 - @spectrum-css/fieldgroup@4.2.5-next.2
 - @spectrum-css/fieldlabel@8.0.0-next.2
 - @spectrum-css/floatingactionbutton@1.2.6-next.2
 - @spectrum-css/helptext@4.1.6-next.2
 - @spectrum-css/icon@6.0.6-next.2
 - @spectrum-css/illustratedmessage@6.1.6-next.2
 - @spectrum-css/infieldbutton@4.2.5-next.2
 - @spectrum-css/inlinealert@7.1.7-next.2
 - @spectrum-css/link@4.2.6-next.2
 - @spectrum-css/logicbutton@3.3.5-next.2
 - @spectrum-css/menu@6.1.6-next.2
 - @spectrum-css/miller@5.1.5-next.2
 - @spectrum-css/modal@4.2.7-next.2
 - @spectrum-css/opacitycheckerboard@1.1.6-next.2
 - @spectrum-css/page@7.1.5-next.2
 - @spectrum-css/pagination@7.1.7-next.2
 - @spectrum-css/picker@7.2.8-next.2
 - @spectrum-css/pickerbutton@4.1.6-next.2
 - @spectrum-css/popover@6.2.6-next.2
 - @spectrum-css/progressbar@3.1.6-next.2
 - @spectrum-css/progresscircle@2.2.4-next.2
 - @spectrum-css/radio@8.1.6-next.2
 - @spectrum-css/rating@4.2.5-next.2
 - @spectrum-css/search@6.2.5-next.2
 - @spectrum-css/sidenav@4.2.5-next.2
 - @spectrum-css/site@4.2.5-next.2
 - @spectrum-css/slider@4.3.6-next.2
 - @spectrum-css/splitview@4.2.5-next.2
 - @spectrum-css/statuslight@6.1.7-next.2
 - @spectrum-css/steplist@4.1.5-next.2
 - @spectrum-css/stepper@5.1.6-next.2
 - @spectrum-css/swatch@5.1.6-next.2
 - @spectrum-css/swatchgroup@2.1.6-next.2
 - @spectrum-css/switch@4.2.6-next.2
 - @spectrum-css/table@5.2.6-next.2
 - @spectrum-css/tabs@4.1.5-next.2
 - @spectrum-css/tag@8.1.6-next.2
 - @spectrum-css/taggroup@4.1.6-next.2
 - @spectrum-css/textfield@6.1.7-next.2
 - @spectrum-css/thumbnail@5.2.5-next.2
 - @spectrum-css/toast@9.1.26-next.2
 - @spectrum-css/tooltip@5.3.6-next.2
 - @spectrum-css/tray@2.2.8-next.2
 - @spectrum-css/treeview@9.2.7-next.2
 - @spectrum-css/typography@5.1.6-next.2
 - @spectrum-css/underlay@3.2.5-next.2
 - @spectrum-css/well@4.1.5-next.2
 - @spectrum-css/generator@2.1.1-next.0
 - @spectrum-css/tokens@14.0.0-next.6
* feat(switch): s2 migration

* chore: remove themes

* fix: animation

* fix: whcm light mode unchecked handle shows

* docs(storybook): align chromatic setup with new standard

* fix: checked corners for non-retina display

* fix: whcm handle selected color

* fix: namespace mods, use semantic tokens instead of global

* fix: handle state colors
* chore(tokens): use spectrum-tokens@13.0.0-beta.30
* chore: add changeset
* chore: release (next)

* docs: reset changes to changelog

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Patrick Fulton <pfulton@adobe.com>
* feat(fieldlabel)!: s2 migration (#2569)

BREAKING CHANGE: uses Spectrum 2 tokens

Also:
* feat(fieldlabel): medium as default and 100 sizing tokens

- Use the correct "100" sized tokens for medium.
- Make medium styles the default, to conform with existing pattern.
  .spectrum-FieldLabel--sizeM class removed as it is no longer
  necessary.

* feat(fieldlabel): use correct font size tokens

The font-size tokens used for large and extra large needed to be shifted
up to match the s2 spec.

* refactor(fieldlabel): remove deprecated mods

Remove renamed mods marked as deprecated for S2 release.

* docs(fieldlabel): spectrum 2 noted in migration notes

Updates some docs and adds migration notes. Clarifies some docs around
"left" and "right", and fixes duplicate ID warnings from examples code.

* docs(fieldlabel): form - docs update and regenerate mods

Add migration guide section to "Form", that currently lives within the
Field label component. Regenerates mods to include the removal of
deprecated renamed mods.

* fix(fieldlabel): rename mod property referencing global token

A mod custom property was referencing a global token instead of the
component itself. Renames this and adds a note to the migration guide.

* feat(fieldlabel): use correct sizing tokens

Use correct tokens for some large and extra large custom properties,
to matching with design.

* style(fieldlabel): applying linter formatting adjustments

Run linter fix on the fieldlabel component, which rearranges a few
declarations.

* docs(fieldlabel): document asterisk icon size

Add documentation about asterisk UI icon sizes specified on the design.
These sizes are already handled in the Storybook template, but were not
really explained to the user.

* feat(fieldlabel): black and white static color variants

Add classes for static colors, as defined on the designs. And adds
examples of these variants to the documentation.

* build(fieldlabel): specify s2 tokens release for dependency

Use Spectrum 2 version of tokens package in dependencies list.

* chore: rebase against main and update components to use prerelease tokens

* chore: run branch through format; update pathing and fix errors

* chore: add changeset for css update

---------

Co-authored-by: Josh Winn <965114+jawinn@users.noreply.github.com>
Co-authored-by: Patrick Fulton <pfulton@adobe.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
mdt2 and others added 2 commits April 26, 2024 19:07
* feat(downstate): docs + implementation for example components

* docs: update mdx

* docs: reorg, stories live within foundations

* docs: decorator for down state dimension tokens

* docs: fix mdx hierarchy console error

* fix: small iconOnly button gets min perspective

* docs: use markdown, update language

* fix: disabled, readonly checkbox doesnt have down state

* chore(button,checkbox): update package versions
* feat: add example gradients for static black and white

Add new gradient backgrounds, displayed for static black and static
white variants. These are used for examples only. This adds CSS custom
properties available globally within Storybook and sets them on the
existing decorator.

* docs(fieldlabel): support static colors decorator in storybook

Change Field label stories that show static black and static white,
so they work with the recently added decorator that changes the main
Storybook background.

* docs(button): adjust static colors template

Adjust static colors template to better handle the static color
decorator and gradients.

* chore(fieldlabel): apply eslint indentation changes to stories
Copy link

changeset-bot bot commented Apr 26, 2024

⚠️ No Changeset found

Latest commit: 50f09e4

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S2 Spectrum 2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants