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: improve error message in useIsSSR() #23750

Merged
merged 1 commit into from Jun 29, 2022

Conversation

layershifter
Copy link
Member

New Behavior

This PR improves an error that can be thrown by useIsSSR() hook to include source and documentation link as was suggested in #23620 (comment).

@layershifter layershifter marked this pull request as ready for review June 28, 2022 14:10
@layershifter layershifter requested a review from a team as a code owner June 28, 2022 14:10
@github-actions github-actions bot added this to the June Project Cycle Q2 2022 milestone Jun 28, 2022
@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 0371306:

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)
global-context
createContext
533 B
341 B
global-context
createContextSelector
554 B
348 B
react-accordion
Accordion (including children components)
77.836 kB
23.683 kB
react-avatar
Avatar
46.757 kB
13.737 kB
react-avatar
AvatarGroup
136.183 kB
40.662 kB
react-avatar
AvatarGroupItem
66.472 kB
19.079 kB
react-badge
Badge
21.653 kB
6.862 kB
react-badge
CounterBadge
22.557 kB
7.172 kB
react-badge
PresenceBadge
23.357 kB
7.099 kB
react-button
Button
35.704 kB
9.499 kB
react-button
CompoundButton
42.777 kB
10.733 kB
react-button
MenuButton
38.279 kB
10.365 kB
react-button
SplitButton
45.612 kB
11.71 kB
react-button
ToggleButton
50.929 kB
10.919 kB
react-card
Card - All
66.957 kB
19.042 kB
react-card
Card
62.113 kB
17.911 kB
react-card
CardFooter
8.461 kB
3.555 kB
react-card
CardHeader
10.026 kB
4.067 kB
react-card
CardPreview
8.562 kB
3.61 kB
react-combobox
Combobox (including child components)
73.69 kB
23.901 kB
react-combobox
Dropdown (including child components)
73.203 kB
23.895 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
189.067 kB
52.144 kB
react-components
react-components: FluentProvider & webLightTheme
31.883 kB
10.516 kB
react-divider
Divider
16.321 kB
5.837 kB
react-image
Image
10.68 kB
4.215 kB
react-input
Input
22.861 kB
7.512 kB
react-label
Label
9.238 kB
3.815 kB
react-link
Link
12.082 kB
4.88 kB
react-menu
Menu (including children components)
116.688 kB
35.475 kB
react-menu
Menu (including selectable components)
119.863 kB
35.96 kB
react-overflow
hooks only
10.839 kB
4.146 kB
react-popover
Popover
104.924 kB
31.964 kB
react-portal
Portal
10.033 kB
3.753 kB
react-positioning
usePositioning
23.839 kB
8.292 kB
react-provider
FluentProvider
14.76 kB
5.596 kB
react-radio
Radio
30.438 kB
10.41 kB
react-radio
RadioGroup
14.319 kB
5.711 kB
react-select
Select
20.002 kB
7.171 kB
react-slider
Slider
26.136 kB
8.387 kB
react-spinbutton
SpinButton
43.378 kB
12.277 kB
react-spinner
Spinner
19.073 kB
6.308 kB
react-switch
Switch
26.578 kB
8.6 kB
react-text
Text - Default
11.572 kB
4.537 kB
react-text
Text - Wrappers
14.882 kB
4.977 kB
react-textarea
Textarea
22.228 kB
7.523 kB
react-tooltip
Tooltip
44.55 kB
15.386 kB
react-utilities
SSRProvider
189 B
161 B
🤖 This report was generated against ba6c5d651559b91c815429c9a9357c4d5a390f3e

@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 1337 1323 5000
Button mount 1003 1009 5000
FluentProvider mount 1710 1763 5000
FluentProviderWithTheme mount 670 663 10
FluentProviderWithTheme virtual-rerender 647 650 10
FluentProviderWithTheme virtual-rerender-with-unmount 675 674 10
MakeStyles mount 2075 2069 50000

@size-auditor
Copy link

size-auditor bot commented Jun 28, 2022

Asset size changes

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

Baseline commit: ba6c5d651559b91c815429c9a9357c4d5a390f3e (build)

@ling1726 ling1726 merged commit a9a6e9a into microsoft:master Jun 29, 2022
marcosmoura pushed a commit to marcosmoura/fluentui that referenced this pull request Jun 29, 2022
…ect-amout-of-items

* master:
  chore: improve error message in useIsSSR() (microsoft#23750)
  fix: Codesandbox exports should use ^9.0.0 (microsoft#23764)
  applying package updates
  Update BadgeColor.stories.tsx (microsoft#23762)
  chore(react-avatar): Adding unit and vr tests for AvatarGroup (microsoft#23549)
  docs(react-avatar): Adding readme and migration guide for AvatarGroup (microsoft#23758)
  update deps after 9.0.1 release (microsoft#23756)
  applying package updates
  fix: Use caret dependency range for Griffel (microsoft#23754)
  Theme designer: minor fixes (microsoft#23741)
  fix: Dependency mismatch after 9.0.0 release (microsoft#23752)
@layershifter layershifter deleted the docs/ssr-warning branch July 1, 2022 07:56
khmakoto pushed a commit to khmakoto/fluentui that referenced this pull request Jul 13, 2022
khmakoto pushed a commit to khmakoto/fluentui that referenced this pull request Jul 13, 2022
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.

None yet

4 participants