Skip to content

Conversation

@kkakroo
Copy link
Contributor

@kkakroo kkakroo commented Apr 24, 2023

Previous Behavior

useOverflowCount was not exported from fluentui/react-components

New Behavior

useOverflowCount is now being exported from fluentui/react-components

Related Issue(s)

@kkakroo kkakroo requested a review from a team as a code owner April 24, 2023 21:53
@fabricteam
Copy link
Collaborator

fabricteam commented Apr 24, 2023

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
65.31 kB
17.931 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
204.5 kB
57.183 kB
react-components
react-components: FluentProvider & webLightTheme
36.086 kB
11.9 kB
react-portal-compat
PortalCompatProvider
6.446 kB
2.186 kB
🤖 This report was generated against 42ecce73761a3bac89a6870dd6d84fde421bfe71

@size-auditor
Copy link

size-auditor bot commented Apr 24, 2023

Asset size changes

⚠️ Insufficient baseline data to detect size changes

Unable to find bundle size details for Baseline commit: 3a6bbb0

Possible causes

  • The baseline build 3a6bbb0 is broken
  • The Size Auditor run for the baseline build 3a6bbb0 was not triggered

Recommendations

  • Please merge your branch for this Pull request with the latest master build and commit your changes once again

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 24, 2023

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 719 699 5000
Button mount 380 378 5000
Field mount 1265 1296 5000
FluentProvider mount 911 911 5000
FluentProviderWithTheme mount 112 120 10
FluentProviderWithTheme virtual-rerender 94 96 10
FluentProviderWithTheme virtual-rerender-with-unmount 101 106 10
InfoButton mount 20 18 5000
MakeStyles mount 1130 1128 50000
Persona mount 2036 1984 5000
SpinButton mount 1580 1574 5000

@ling1726
Copy link
Contributor

@kkakroo thanks for contributing. To make CI pass please do the following:

  • Run yarn build --to @fluentui/react-components to update the API mark down file
  • Run yarn change to add a changelog entry

@kkakroo
Copy link
Contributor Author

kkakroo commented Apr 25, 2023

Hi @ling1726, have updated the api markdown file but yarn change did not add a changelog entry again.

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 25, 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 c88fd14:

Sandbox Source
@fluentui/react 8 starter Configuration
@fluentui/react-components 9 starter Configuration
cranky-sid-8t39p6 Issue #27657

@ling1726 ling1726 merged commit d80eca9 into microsoft:master Apr 25, 2023
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Apr 27, 2023
* master:
  chore: enforce files naming to use .styles.ts [cxe-prg files] (microsoft#27707)
  chore: migrate to NodeJS v16 (microsoft#27711)
  remove react-avatar-context package (microsoft#27709)
  test: migrate all v9 libraries to use new conformance setup which improves test execution by approx 46% (microsoft#27669)
  chore: Add caret to react-datepicker-compat dependencies (microsoft#27671)
  test(fluentui/react): fix initial global leaks that were causing OOM issues (microsoft#27661)
  chore(react-utilities): simplifies useControllableState hook internals (microsoft#27702)
  chore: enforce files naming to use .styles.ts (microsoft#27698)
  feat: adds lazy loading example (microsoft#27587)
  feat: react-combobox space conditionally inserts character when freeform is true (microsoft#27025)
  chore(react-tree): updates useOpenItemsState internals (microsoft#27697)
  feat(tools): add conformance setup migration to migrate-converged-pkg generator (microsoft#27668)
  test: use `isolatedModules` for all ts-jest configs to lower memory footprint on CI (microsoft#27670)
  chore: bump @griffel/eslint-plugin & enable @griffel/hook-naming (microsoft#27687)
  feat(react-tree): makes useFlatTree generic (microsoft#27682)
  Enable export to codepen for charting library (microsoft#27539)
  Overflow: added useOverflowCount to exports (microsoft#27678)
  fix: Menu should not steal focus on re-render (microsoft#27688)
  feat(react-conformance): add new TS config api to be able to specify configName and configDir (microsoft#27664)
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.

[Bug]: useOverflowCount is not exported in fluentui/react-components

3 participants