Skip to content

keyboard-keys: remove star exports#22681

Merged
spmonahan merged 1 commit intomicrosoft:masterfrom
spmonahan:keyboard-keys/star-exports
May 2, 2022
Merged

keyboard-keys: remove star exports#22681
spmonahan merged 1 commit intomicrosoft:masterfrom
spmonahan:keyboard-keys/star-exports

Conversation

@spmonahan
Copy link
Contributor

Current Behavior

keyboard-keys uses star exports

New Behavior

keyboard-keys does not use star exports

Related Issue(s)

#22099

Removes star exports from the public API per:
microsoft#22099
@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 2b51cac:

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

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-accordion
Accordion (including children components)
73.754 kB
22.494 kB
react-badge
Badge
20.965 kB
6.605 kB
react-badge
CounterBadge
21.918 kB
6.919 kB
react-badge
PresenceBadge
22.005 kB
6.582 kB
react-card
Card - All
57.884 kB
16.524 kB
react-card
Card
53.169 kB
15.238 kB
react-card
CardFooter
7.756 kB
3.299 kB
react-card
CardHeader
9.321 kB
3.816 kB
react-card
CardPreview
7.728 kB
3.323 kB
react-combobox
Combobox
60.564 kB
20.751 kB
react-divider
Divider
15.455 kB
5.57 kB
react-image
Image
10.179 kB
3.995 kB
react-input
Input
21.775 kB
7.228 kB
react-label
Label
8.441 kB
3.541 kB
react-link
Link
11.176 kB
4.545 kB
react-positioning
usePopper
23.21 kB
8.084 kB
react-priority-overflow
hooks only
10.792 kB
4.125 kB
react-provider
FluentProvider
14.079 kB
5.274 kB
react-spinbutton
SpinButton
41.955 kB
11.856 kB
react-spinner
Spinner
17.224 kB
5.856 kB
react-switch
Switch
24.333 kB
8.019 kB
react-text
Text - Default
10.867 kB
4.269 kB
react-text
Text - Wrappers
14.183 kB
4.61 kB
react-textarea
Textarea
20.672 kB
7.07 kB
react-tooltip
Tooltip
42.907 kB
14.761 kB
react-utilities
SSRProvider
189 B
161 B
🤖 This report was generated against e74b180720cb9aec8df21753a55449849041af82

@size-auditor
Copy link

size-auditor bot commented Apr 27, 2022

Asset size changes

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

Baseline commit: e74b180720cb9aec8df21753a55449849041af82 (build)

@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 865 914 5000
Button mount 481 513 5000
FluentProvider mount 1787 1845 5000
FluentProviderWithTheme mount 232 246 10
FluentProviderWithTheme virtual-rerender 227 200 10
FluentProviderWithTheme virtual-rerender-with-unmount 291 263 10
MakeStyles mount 1483 1432 50000

@spmonahan spmonahan merged commit dd474c7 into microsoft:master May 2, 2022
marwan38 pushed a commit to marwan38/fluentui that referenced this pull request Jun 13, 2022
Removes star exports from the public API per:
microsoft#22099
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