Skip to content

Conversation

@layershifter
Copy link
Member

Previous Behavior

PortalCompatProvider can handle only one fui-FluentProvider* class from a context.

New Behavior

PortalCompatProvider can handle multiple fui-FluentProvider* classes from a context. This a customer scenario when additional tokens are added with useFluentProviderThemeStyleTag().

@layershifter layershifter requested a review from a team as a code owner September 30, 2023 06:38
@fabricteam
Copy link
Collaborator

📊 Bundle size report

Package & Exports Baseline (minified/GZIP) PR Change
react-portal-compat
PortalCompatProvider
6.48 kB
2.203 kB
6.503 kB
2.22 kB
23 B
17 B

🤖 This report was generated against d54dc8a1c10f8123413fe018e607cee9cffe2199

@codesandbox-ci
Copy link

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 1f41a1f:

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

@size-auditor
Copy link

size-auditor bot commented Sep 30, 2023

Asset size changes

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

Baseline commit: d54dc8a1c10f8123413fe018e607cee9cffe2199 (build)

@layershifter layershifter merged commit 18ddec9 into microsoft:master Oct 2, 2023
@layershifter layershifter deleted the fix/portal-compat-multiple branch October 2, 2023 08:14
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 4, 2023
* master: (146 commits)
  remove a11y test ux from theme designer (microsoft#29379)
  feat: preview release (microsoft#29377)
  Fixed divider in the tooltip (microsoft#29357)
  fix (microsoft#29376)
  docs: Initial documentation pass (microsoft#29372)
  feat: Integrate Announce context (microsoft#29362)
  chore(react-tags): stable release (microsoft#29355)
  chore: delegate focus outline to layout components (microsoft#29293)
  Vertical bar chart line bug (microsoft#29299)
  applying package updates
  Adding axis title examples for line, area, vertical bar and vertical stacked bar charts (microsoft#29254)
  A11y: Pagination: Add tooltip for all icon buttons  (microsoft#29155)
  A11y: Fix aria attribute for page number and add correct role (microsoft#29349)
  feat: Implement motion for MessageBar (microsoft#29339)
  feat: creates getIntrinsicElementProps to replace getNativeElementProps on slots creation (microsoft#29310)
  chore: replace makeStyles with makeResetStyles (microsoft#29338)
  applying package updates
  ci(github/policies): remove bot rules that created and added `Component/Package $1`: labels on PRs (microsoft#29301)
  fix: handle multiple classes in PortalCompatProvider (microsoft#29351)
  applying package updates
  ...
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 4, 2023
* master: (49 commits)
  remove a11y test ux from theme designer (microsoft#29379)
  feat: preview release (microsoft#29377)
  Fixed divider in the tooltip (microsoft#29357)
  fix (microsoft#29376)
  docs: Initial documentation pass (microsoft#29372)
  feat: Integrate Announce context (microsoft#29362)
  chore(react-tags): stable release (microsoft#29355)
  chore: delegate focus outline to layout components (microsoft#29293)
  Vertical bar chart line bug (microsoft#29299)
  applying package updates
  Adding axis title examples for line, area, vertical bar and vertical stacked bar charts (microsoft#29254)
  A11y: Pagination: Add tooltip for all icon buttons  (microsoft#29155)
  A11y: Fix aria attribute for page number and add correct role (microsoft#29349)
  feat: Implement motion for MessageBar (microsoft#29339)
  feat: creates getIntrinsicElementProps to replace getNativeElementProps on slots creation (microsoft#29310)
  chore: replace makeStyles with makeResetStyles (microsoft#29338)
  applying package updates
  ci(github/policies): remove bot rules that created and added `Component/Package $1`: labels on PRs (microsoft#29301)
  fix: handle multiple classes in PortalCompatProvider (microsoft#29351)
  applying package updates
  ...
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