Skip to content

Conversation

@zoopmaster
Copy link
Contributor

@zoopmaster zoopmaster commented Jan 29, 2025

Task

OP #170

Why?

In an effort to move away from sass, following a strict BEM structure in needed for Alerts.

What Changed

What changed in this PR?

  • Refactor accordion.scss and docs
  • Refactor avatar.scss and docs
  • Refactor breadcrumbs.scss and docs
  • Refactor pagination.scss and docs
  • Refactor spinner.scss and docs
  • Refactor tooltip.scss and docs
  • Refactor switch.scss and docs
  • Refactor tab.scss and docs
  • Refactor text-pair.scss and docs
  • Refactor divider.scss and docs

Sanity Check

  • [ ] Have you updated any usage of changed tokens?
  • Have you updated the docs with any component changes?
  • [ ] Have you updated the dependency graph with any component changes?
  • Have you run linters?
  • Have you run prettier?
  • Have you tried building the css?
  • Have you tried building storybook?
  • Do you need to update the package version?

@zoopmaster zoopmaster added Breaking Change This will cause problems so be sure to indicate that in the release notes Components Changes to a component labels Jan 29, 2025
@zoopmaster zoopmaster self-assigned this Jan 29, 2025
@linear
Copy link

linear bot commented Jan 29, 2025

@Jeremy-Walton Jeremy-Walton merged commit 58e37bc into main Jan 30, 2025
1 check passed
@Jeremy-Walton Jeremy-Walton deleted the op-170-refactor-simple-components branch January 30, 2025 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaking Change This will cause problems so be sure to indicate that in the release notes Components Changes to a component

Development

Successfully merging this pull request may close these issues.

4 participants