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

feat: removes dependency on fast-foundation templates and classes. #30090

Merged

Conversation

eljefe223
Copy link
Contributor

@eljefe223 eljefe223 commented Dec 15, 2023

Previous Behavior

New Behavior

Removes dependency on fast-foundation classes and templates.

Related Issue(s)

  • Fixes #

@fabricteam
Copy link
Collaborator

fabricteam commented Dec 15, 2023

📊 Bundle size report

🤖 This report was generated against 4909fe2c8e9fb9029b5e7fa4d5aa73d299cf089f

Copy link

size-auditor bot commented Dec 15, 2023

Asset size changes

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

Baseline commit: 12da47698c9121c0345c2f0db481cbae7a91b66e (build)

Copy link

codesandbox-ci bot commented Dec 15, 2023

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 da1d0a5:

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

@eljefe223 eljefe223 marked this pull request as ready for review January 9, 2024 19:21
@eljefe223 eljefe223 requested a review from a team as a code owner January 9, 2024 19:21
@eljefe223 eljefe223 enabled auto-merge (squash) January 31, 2024 23:26
@eljefe223 eljefe223 force-pushed the users/jes/remove-foundation branch 5 times, most recently from b5e5cfb to 4138edc Compare February 9, 2024 03:53
@eljefe223 eljefe223 merged commit 8fd063a into microsoft:web-components-v3 Feb 9, 2024
15 of 19 checks passed
@fabricteam
Copy link
Collaborator

🕵 fluentui-web-components-v3 No visual regressions between this PR and main

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