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

Audit and consolidate styles between carbon-components and @carbon/styles #9805

Closed
47 tasks done
Tracked by #10316
tay1orjones opened this issue Oct 5, 2021 · 3 comments
Closed
47 tasks done
Tracked by #10316
Assignees
Labels
planning: umbrella Umbrella issues, surfaced in Projects views role: dev 🤖 version: 11 Issues pertaining to Carbon v11

Comments

@tay1orjones
Copy link
Member

tay1orjones commented Oct 5, 2021

Leading up to v11 we have migrated styles from carbon-components over into @carbon/styles and in the process converted the styles to be configured as sass modules.

Since then, there have been bug fixes and new features made to carbon-components changing the styles. These changes need to be found and ensure they have been ported over to @carbon/styles. In addition there have been purposeful changes made in @carbon/styles for v11 that are breaking changes and should not be ported back to carbon-components.

Moving forward:

  • Changes to carbon-components should also be made to @carbon/styles, or tracked here to be migrated to @carbon/styles at a later date during a final audit to take place closer to v11 stable release.
  • Purposeful (breaking) changes made only to @carbon/styles for v11 can be noted on this issue in attempt to ensure these changes are retained during the final audit.

Steps to take for each component are outlined in the comment below ↓

Checklist

@tay1orjones
Copy link
Member Author

It appears the progress Indicator styles are incorrect in v11

@tay1orjones
Copy link
Member Author

tay1orjones commented Jan 31, 2022

Steps for each component

  1. Find differences between v10/v11 components
    • Diff the two style files, identify any meaningful differences other than tokens/sass modules syntax
    • Visually review the component in existing storybook and new storybook
  2. Identify if the change should be present in v10, v11, or both.
    • Look at git history of those changes
    • Review related PRs
    • Ask PR authors?
  3. Mirror changes from v10 to v11 if applicable

@tay1orjones
Copy link
Member Author

Complete 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
planning: umbrella Umbrella issues, surfaced in Projects views role: dev 🤖 version: 11 Issues pertaining to Carbon v11
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants