Skip to content

feat: cleanup Carbon override stylesheet#190

Merged
MCatherine1994 merged 8 commits intomainfrom
clean-stylesheet
Mar 6, 2026
Merged

feat: cleanup Carbon override stylesheet#190
MCatherine1994 merged 8 commits intomainfrom
clean-stylesheet

Conversation

@MCatherine1994
Copy link
Collaborator

@MCatherine1994 MCatherine1994 commented Sep 9, 2025

Following the documentation in this issue, I cleaned up the existing Carbon override stylesheet, regrouped the style changes by component, and added comments for each section. There was no style change, just cleanup and refactoring.

@craigyu craigyu requested a review from Copilot September 18, 2025 22:01
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the Carbon override stylesheet to improve organization and maintainability. The cleanup includes grouping styles by component, adding descriptive comments for each section, and reorganizing the code structure without making any functional style changes.

  • Regrouped CSS overrides by Carbon component type with clear section headers
  • Added detailed comments explaining the styling changes for each component
  • Updated font file paths to use newer BC Sans font variants with light weight support

Reviewed Changes

Copilot reviewed 3 out of 16 changed files in this pull request and generated 2 comments.

File Description
style-sheets/overrides.scss Complete reorganization of CSS overrides with component-based grouping and comprehensive commenting
design-tokens/type-family.scss Updated font file paths and added light font weight support
README.md Added new React usage documentation with code examples

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@craigyu craigyu self-requested a review September 18, 2025 22:12
@MCatherine1994 MCatherine1994 merged commit 6e61514 into main Mar 6, 2026
3 checks passed
@MCatherine1994 MCatherine1994 deleted the clean-stylesheet branch March 6, 2026 23:25
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.

3 participants