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: make utils a package export #31865

Merged
merged 4 commits into from
Jun 28, 2024

Conversation

mohamedmansour
Copy link
Contributor

@mohamedmansour mohamedmansour commented Jun 28, 2024

Previous Behavior

Previously we used to use @microsoft/fast-foundation/utilities.js and since fast-foundation is now merged into fluentui/web-components we don't want to carry both packages, so exporting the utilities.js is needed (so bundle bloat doesn't happen) due to sideEffected packages.

New Behavior

This creates @fluentui/web-components/utilities.js

@mohamedmansour mohamedmansour force-pushed the mmansour-export-utils branch from 7314e8d to 8affba2 Compare June 28, 2024 19:33
@mohamedmansour mohamedmansour force-pushed the mmansour-export-utils branch from 8affba2 to 9eff123 Compare June 28, 2024 19:34
@mohamedmansour mohamedmansour marked this pull request as ready for review June 28, 2024 19:36
@mohamedmansour mohamedmansour requested a review from a team as a code owner June 28, 2024 19:36
@chrisdholt chrisdholt enabled auto-merge (squash) June 28, 2024 22:15
@chrisdholt chrisdholt merged commit 286ef61 into microsoft:master Jun 28, 2024
10 checks passed
@mohamedmansour mohamedmansour deleted the mmansour-export-utils branch June 28, 2024 23:01
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jul 3, 2024
* master: (45 commits)
  Added unit testcases for sparkline chart. (microsoft#31843)
  release: applying package updates - react v8
  release: applying package updates - web-components
  fix(web-components): fix enterkey interaction on menu (microsoft#31894)
  chore(web-components): expose field to package exports (microsoft#31904)
  fix(web-components): slotted elements within fluent-text-input can't be focused (microsoft#31903)
  fix(web-components): fix menu positioning in firefox (microsoft#31895)
  chore: bump @types/node types to v20 and @tsconfig to v18 (microsoft#31510)
  chore: migrate to nx 19.2.3 (microsoft#31828)
  chore: fix install verdacio command (microsoft#31897)
  fix(scripts-prettier): escape file names that are being passed to prettier bin via shell (microsoft#31889)
  release: applying package updates - react-components
  feat:(docs) Adds pattern doc on Basic Recomposition (microsoft#31861)
  chore:(docs) Archive the wiki 🎉 (microsoft#31202)
  feat:(react-nav-preview) Adds NavDivider implementation (microsoft#31848)
  ci(.github): resolve due_on UTC time creation issues (microsoft#31890)
  release: applying package updates - web-components
  fix: update internal usage of useModalAttributes to use legacyTrapFocus by default (microsoft#31801)
  chore: make utils a package export (microsoft#31865)
  fix: add live region around scroll index update (microsoft#31863)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants