Skip to content

Conversation

pfulton
Copy link
Collaborator

@pfulton pfulton commented Sep 6, 2023

BREAKING CHANGE: migrates the Page component to use @adobe/spectrum-tokens

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.

Regression testing

Validate:

  1. A legacy documentation page (such as accordion), including:
  • The page renders correctly
  • The page is accessible
  • The page is responsive
  1. A migrated documentation page (such as action group), including:
  • The page renders correctly
  • The page is accessible
  • The page is responsive

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

@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2023

🚀 Deployed on https://pr-2150--spectrum-css.netlify.app

@github-actions github-actions bot temporarily deployed to pull request September 6, 2023 17:33 Inactive
Copy link
Contributor

@mlogsdon18 mlogsdon18 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Easy enough! Looks good to me

BREAKING CHANGE: migrates the Page component to use `@adobe/spectrum-tokens`
@pfulton pfulton force-pushed the feat-migrate-page-component branch from 77fc9e9 to e98bcc4 Compare September 6, 2023 17:55
@github-actions github-actions bot temporarily deployed to pull request September 6, 2023 18:01 Inactive
@pfulton pfulton merged commit bdf535b into main Sep 6, 2023
@pfulton pfulton deleted the feat-migrate-page-component branch September 6, 2023 18:02
Rajdeepc pushed a commit that referenced this pull request Sep 11, 2023
BREAKING CHANGE: migrates the Page component to use `@adobe/spectrum-tokens`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants