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

Update vr screenshotdiff lib to accept vr host url as param #25772

Merged
merged 119 commits into from
Nov 29, 2022

Conversation

sunilsurana
Copy link
Member

Previous Behavior

New Behavior

Update vr screenshotdiff lib to accept vr host url as param

Related Issue(s)

  • Fixes #

@size-auditor
Copy link

size-auditor bot commented Nov 24, 2022

Asset size changes

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

Baseline commit: a1ed5fe8f610c82cc319add155655e9f66b3ff8e (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 24, 2022

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 1651 1687 5000
Button mount 1153 1149 5000
FluentProvider mount 1949 1912 5000
FluentProviderWithTheme mount 728 737 10
FluentProviderWithTheme virtual-rerender 671 662 10
FluentProviderWithTheme virtual-rerender-with-unmount 710 700 10
MakeStyles mount 2291 2276 50000
SpinButton mount 3268 3278 5000

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 24, 2022

🕵 fluentuiv8 Open the Visual Regressions report to inspect the 2 screenshots

✅ There was 0 screenshots added, 2 screenshots removed, 1037 screenshots unchanged, 0 screenshots with different dimensions and 0 screenshots with visible difference.

unknown 2 screenshots
Image Name Diff(in Pixels) Image Type
Pivot - Overflow.Tabs - RTL.Narrow - Last tab selected.chromium.png 0 Removed
Pivot - Overflow.Tabs - RTL.Narrow - Overflow menu.chromium.png 0 Removed

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 24, 2022

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
global-context
createContext
533 B
341 B
global-context
createContextSelector
554 B
348 B
priority-overflow
createOverflowManager
3.153 kB
1.299 kB
react-accordion
Accordion (including children components)
79.294 kB
24.443 kB
react-alert
Alert
86.891 kB
21.708 kB
react-avatar
Avatar
51.076 kB
14.285 kB
react-avatar
AvatarGroup
15.006 kB
6.009 kB
react-avatar
AvatarGroupItem
65.654 kB
18.382 kB
react-badge
Badge
23.357 kB
7.361 kB
react-badge
CounterBadge
24.247 kB
7.643 kB
react-badge
PresenceBadge
24.001 kB
7.049 kB
react-button
Button
36.86 kB
9.913 kB
react-button
CompoundButton
43.885 kB
11.131 kB
react-button
MenuButton
41.537 kB
11.187 kB
react-button
SplitButton
48.983 kB
12.588 kB
react-button
ToggleButton
52.685 kB
11.415 kB
react-card
Card - All
71.873 kB
20.936 kB
react-card
Card
67.651 kB
19.892 kB
react-card
CardFooter
8.617 kB
3.62 kB
react-card
CardHeader
9.792 kB
3.995 kB
react-card
CardPreview
8.718 kB
3.674 kB
react-combobox
Combobox (including child components)
78.244 kB
25.015 kB
react-combobox
Dropdown (including child components)
77.531 kB
24.927 kB
react-components
react-components: Button, FluentProvider & webLightTheme
62.975 kB
17.687 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
190.411 kB
53.077 kB
react-components
react-components: FluentProvider & webLightTheme
33.48 kB
11.037 kB
react-dialog
Dialog (including children components)
83.036 kB
24.754 kB
react-divider
Divider
16.515 kB
5.921 kB
react-image
Image
10.836 kB
4.283 kB
react-infobutton
InfoButton
117.922 kB
35.391 kB
react-input
Input
24.194 kB
7.89 kB
react-label
Label
9.394 kB
3.88 kB
react-link
Link
11.862 kB
4.885 kB
react-menu
Menu (including children components)
117.61 kB
36.287 kB
react-menu
Menu (including selectable components)
120.679 kB
36.82 kB
react-overflow
hooks only
11.004 kB
4.188 kB
react-persona
Persona
57.131 kB
15.951 kB
react-popover
Popover
103.666 kB
31.858 kB
react-portal
Portal
10.495 kB
3.851 kB
react-portal-compat
PortalCompatProvider
5.857 kB
1.978 kB
react-positioning
usePositioning
19.826 kB
7.417 kB
react-progress
Progress
13.536 kB
5.107 kB
react-provider
FluentProvider
15.817 kB
5.905 kB
react-radio
Radio
36.446 kB
12.123 kB
react-radio
RadioGroup
14.304 kB
5.72 kB
react-select
Select
23.474 kB
8.017 kB
react-slider
Slider
32.118 kB
10.192 kB
react-spinbutton
SpinButton
44.502 kB
12.812 kB
react-spinner
Spinner
20.013 kB
6.446 kB
react-switch
Switch
33.453 kB
10.581 kB
react-text
Text - Default
11.838 kB
4.625 kB
react-text
Text - Wrappers
15.148 kB
5.06 kB
react-textarea
Textarea
25.733 kB
8.328 kB
react-theme
Single theme token import
69 B
89 B
react-theme
Teams: all themes
29.722 kB
6.462 kB
react-theme
Teams: Light theme
17.51 kB
5.075 kB
react-tooltip
Tooltip
42.032 kB
14.739 kB
react-utilities
SSRProvider
180 B
159 B
🤖 This report was generated against a1ed5fe8f610c82cc319add155655e9f66b3ff8e

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 24, 2022

Perf Analysis (@fluentui/react-northstar)

Perf tests with no regressions
Scenario Current PR Ticks Baseline Ticks Ratio
ChatDuplicateMessagesPerf.default 224 205 1.09:1
FlexMinimalPerf.default 258 241 1.07:1
TableMinimalPerf.default 367 345 1.06:1
TextMinimalPerf.default 313 296 1.06:1
DividerMinimalPerf.default 322 308 1.05:1
PortalMinimalPerf.default 143 136 1.05:1
AvatarMinimalPerf.default 165 159 1.04:1
DropdownManyItemsPerf.default 562 538 1.04:1
GridMinimalPerf.default 305 294 1.04:1
AlertMinimalPerf.default 225 218 1.03:1
BoxMinimalPerf.default 301 292 1.03:1
ChatWithPopoverPerf.default 302 294 1.03:1
ImageMinimalPerf.default 346 337 1.03:1
ListWith60ListItems.default 507 493 1.03:1
TreeWith60ListItems.default 125 121 1.03:1
AnimationMinimalPerf.default 470 462 1.02:1
ButtonSlotsPerf.default 434 426 1.02:1
CardMinimalPerf.default 468 460 1.02:1
CarouselMinimalPerf.default 359 351 1.02:1
ChatMinimalPerf.default 641 631 1.02:1
ItemLayoutMinimalPerf.default 980 965 1.02:1
LayoutMinimalPerf.default 317 310 1.02:1
MenuButtonMinimalPerf.default 1363 1341 1.02:1
RadioGroupMinimalPerf.default 393 385 1.02:1
StatusMinimalPerf.default 615 604 1.02:1
HeaderMinimalPerf.default 314 312 1.01:1
LabelMinimalPerf.default 346 341 1.01:1
MenuMinimalPerf.default 729 724 1.01:1
RosterPerf.default 1719 1704 1.01:1
PopupMinimalPerf.default 563 558 1.01:1
ReactionMinimalPerf.default 336 333 1.01:1
CustomToolbarPrototype.default 2166 2153 1.01:1
ToolbarMinimalPerf.default 802 794 1.01:1
AccordionMinimalPerf.default 120 120 1:1
AttachmentMinimalPerf.default 128 128 1:1
AttachmentSlotsPerf.default 880 879 1:1
ButtonOverridesMissPerf.default 1030 1028 1:1
CheckboxMinimalPerf.default 1534 1533 1:1
DialogMinimalPerf.default 687 685 1:1
DropdownMinimalPerf.default 2186 2176 1:1
EmbedMinimalPerf.default 2649 2651 1:1
HeaderSlotsPerf.default 666 665 1:1
InputMinimalPerf.default 858 858 1:1
ListCommonPerf.default 520 521 1:1
ProviderMinimalPerf.default 319 319 1:1
SegmentMinimalPerf.default 300 299 1:1
SkeletonMinimalPerf.default 304 304 1:1
SliderMinimalPerf.default 1238 1232 1:1
SplitButtonMinimalPerf.default 3289 3276 1:1
TableManyItemsPerf.default 1575 1578 1:1
TextAreaMinimalPerf.default 415 414 1:1
TooltipMinimalPerf.default 1886 1894 1:1
TreeMinimalPerf.default 696 697 1:1
VideoMinimalPerf.default 616 619 1:1
DatepickerMinimalPerf.default 4591 4657 0.99:1
FormMinimalPerf.default 330 332 0.99:1
ListMinimalPerf.default 450 455 0.99:1
ListNestedPerf.default 465 471 0.99:1
ProviderMergeThemesPerf.default 980 988 0.99:1
IconMinimalPerf.default 557 568 0.98:1
ButtonMinimalPerf.default 132 136 0.97:1
LoaderMinimalPerf.default 267 277 0.96:1
RefMinimalPerf.default 182 191 0.95:1

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 24, 2022

🕵 fluentuiv9 Open the Visual Regressions report to inspect the 2 screenshots

✅ There was 0 screenshots added, 0 screenshots removed, 1749 screenshots unchanged, 0 screenshots with different dimensions and 2 screenshots with visible difference.

unknown 2 screenshots
Image Name Diff(in Pixels) Image Type
Avatar Converged.size+active+badge.normal.chromium.png 2 Changed
Avatar Converged.size+active+ring-shadow.normal.chromium.png 8 Changed

@fabricteam
Copy link
Collaborator

fabricteam commented Nov 24, 2022

Perf Analysis (@fluentui/react)

Scenario Render type Master Ticks PR Ticks Iterations Status
GroupedList mount 1798 2082 2 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
BaseButton mount 1176 1153 5000
Breadcrumb mount 2919 3004 1000
Checkbox mount 2525 2580 5000
CheckboxBase mount 2286 2188 5000
ChoiceGroup mount 4168 4117 5000
ComboBox mount 1240 1224 1000
CommandBar mount 8781 9146 1000
ContextualMenu mount 10334 10224 1000
DefaultButton mount 1352 1286 5000
DetailsRow mount 3456 3366 5000
DetailsRowFast mount 3235 3278 5000
DetailsRowNoStyles mount 3132 3403 5000
Dialog mount 2817 2903 1000
DocumentCardTitle mount 548 524 1000
Dropdown mount 3091 2940 5000
FocusTrapZone mount 1985 1930 5000
FocusZone mount 1897 1718 5000
GroupedList mount 1798 2082 2 Possible regression
GroupedList virtual-rerender 1094 1065 2
GroupedList virtual-rerender-with-unmount 1538 1518 2
GroupedListV2 mount 536 497 2
GroupedListV2 virtual-rerender 516 560 2
GroupedListV2 virtual-rerender-with-unmount 549 561 2
IconButton mount 1828 1746 5000
Label mount 697 723 5000
Layer mount 4108 4460 5000
Link mount 807 746 5000
MenuButton mount 1644 1687 5000
MessageBar mount 2304 2297 5000
Nav mount 3061 3063 1000
OverflowSet mount 1226 1313 5000
Panel mount 2369 2398 1000
Persona mount 1173 1208 1000
Pivot mount 1476 1605 1000
PrimaryButton mount 1502 1447 5000
Rating mount 6695 6627 5000
SearchBox mount 1476 1493 5000
Shimmer mount 2841 2667 5000
Slider mount 1918 1863 5000
SpinButton mount 4363 4618 5000
Spinner mount 766 746 5000
SplitButton mount 2954 2947 5000
Stack mount 806 804 5000
StackWithIntrinsicChildren mount 2381 2427 5000
StackWithTextChildren mount 4556 4835 5000
SwatchColorPicker mount 9796 9528 5000
TagPicker mount 2514 2444 5000
TeachingBubble mount 85532 82683 5000
Text mount 715 872 5000
TextField mount 1410 1437 5000
ThemeProvider mount 1483 1461 5000
ThemeProvider virtual-rerender 1030 1037 5000
ThemeProvider virtual-rerender-with-unmount 2051 2056 5000
Toggle mount 1070 1011 5000
buttonNative mount 536 502 5000

@sunilsurana sunilsurana reopened this Nov 24, 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 b20e14f:

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

@sunilsurana sunilsurana merged commit b9ef910 into microsoft:master Nov 29, 2022
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Nov 29, 2022
* master: (34 commits)
  chore(react-tooltip): migrate to new package structure (microsoft#25818)
  chore(react-field): migrate to new package structure (microsoft#25817)
  Update vr screenshotdiff lib to accept vr host url as param (microsoft#25772)
  feat(scripts): enable strict checking for additional sub-folders(packages) v4 (microsoft#25710)
  fix(tools): bump norhtstar packages v9 deps on dep mismatch resolution (microsoft#25806)
  feat: remove react-storybook and replace its functionality via standard react-storybook-addon package (microsoft#25786)
  applying package updates
  chore(react-spinbutton): migrate to new package structure (microsoft#25813)
  chore(react-spinner): migrate to new package structure (microsoft#25814)
  chore(react-provider): migrate to new package structure (microsoft#25809)
  chore(react-radio, shared-contexts): migrate to new package structure (microsoft#25810)
  chore(react-theme): migrate to new package structure (microsoft#25812)
  docs: add Fluent UI Insights EP04 to README (microsoft#25775)
  chore(react-migration-v8-v9): use same build process/setup as v9/ts-solution packages (microsoft#25679)
  docs: Improves `Table` documentation (microsoft#25787)
  feat: improve react-18 tests (microsoft#25758)
  docs: Add examples for DataGrid (microsoft#25783)
  chore(react-tree): scaffold TreeItemLayout (microsoft#25781)
  perf: make ts-minbar test compilation faster and asset preparation simpler (microsoft#25754)
  chore: creates TreeItem and basic Tree (microsoft#25742)
  ...
@sunilsurana sunilsurana deleted the updatescreenshotdifflib branch December 1, 2022 07:03
Hotell pushed a commit to Hotell/fluentui that referenced this pull request Feb 9, 2023
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.

6 participants