Skip to content

fix(react-theme-sass): Remove color palette tokens which are no longer present in react-theme - #23691

Merged
miroslavstastny merged 2 commits into
microsoft:masterfrom
miroslavstastny:feat/reduce-shared-colors-scss
Jun 23, 2022
Merged

fix(react-theme-sass): Remove color palette tokens which are no longer present in react-theme#23691
miroslavstastny merged 2 commits into
microsoft:masterfrom
miroslavstastny:feat/reduce-shared-colors-scss

Conversation

@miroslavstastny

Copy link
Copy Markdown
Member

Follow-up for #23608 which removed some colorPalette tokens from react-theme. This PR removes those tokens from react-theme-sass

@fabricteam

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against 18a0c05530dff13ff19e69652bf4eb6c47ecaa3c

@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 57de709:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration

@size-auditor

size-auditor Bot commented Jun 23, 2022

Copy link
Copy Markdown

Asset size changes

Size Auditor did not detect a change in bundle size for any component!

Baseline commit: 18a0c05530dff13ff19e69652bf4eb6c47ecaa3c (build)

@miroslavstastny
miroslavstastny merged commit 03e380f into microsoft:master Jun 23, 2022
@miroslavstastny
miroslavstastny deleted the feat/reduce-shared-colors-scss branch June 23, 2022 16:27
rohitpagariya pushed a commit to rohitpagariya/fluentui that referenced this pull request Jun 28, 2022
…r present in react-theme (microsoft#23691)

* fix(react-theme-sass): Remove color palette tokens which are no longer present in react-theme

* change file
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.

4 participants