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

chore(charts): fix import #33381

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

layershifter
Copy link
Member

@layershifter layershifter commented Dec 2, 2024

Previous Behavior

makeStyles() is imported from @fluentui/react-components:

Build is broken:

image

There is also no dependency declared:

"@fluentui/react-button": "^9.3.95",
"@fluentui/react-jsx-runtime": "^9.0.46",
"@fluentui/react-overflow": "^9.2.1",
"@fluentui/react-popover": "^9.9.25",
"@fluentui/react-shared-contexts": "^9.21.0",
"@fluentui/react-tabster": "^9.23.0",
"@fluentui/react-theme": "^9.1.22",
"@fluentui/react-tooltip": "^9.4.43",
"@fluentui/react-utilities": "^9.18.17",
"@griffel/react": "^1.5.22",

New Behavior

makeStyles() is imported from @griffel/react.

Related Issue(s)

Breaks my PR #33380 😭

@layershifter layershifter requested a review from a team as a code owner December 2, 2024 12:03
Copy link

github-actions bot commented Dec 2, 2024

📊 Bundle size report

✅ No changes found

@layershifter layershifter enabled auto-merge (squash) December 2, 2024 12:10
@layershifter layershifter merged commit 7e9766f into microsoft:master Dec 2, 2024
13 of 15 checks passed
@layershifter layershifter deleted the fix/build-charts branch December 2, 2024 12:12
Copy link

github-actions bot commented Dec 2, 2024

Pull request demo site: URL

@@ -0,0 +1,7 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

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

🕵🏾‍♀️ visual regressions to review in the fluentuiv9 Visual Regression Report

Avatar Converged 2 screenshots
Image Name Diff(in Pixels) Image Type
Avatar Converged.badgeMask.chromium.png 6 Changed
Avatar Converged.Badge Mask RTL.chromium.png 2 Changed

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