Skip to content

Conversation

@tomi-msft
Copy link
Contributor

@tomi-msft tomi-msft commented Apr 21, 2023

This PR smooths out the wave and pulse animations for Skeleton

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 21, 2023

Perf Analysis (@fluentui/react-components)

No significant results to display.

All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 855 842 5000
Button mount 489 466 5000
Field mount 1561 1509 5000
FluentProvider mount 1097 1137 5000
FluentProviderWithTheme mount 159 142 10
FluentProviderWithTheme virtual-rerender 121 112 10
FluentProviderWithTheme virtual-rerender-with-unmount 137 134 10
InfoButton mount 25 27 5000
MakeStyles mount 1449 1419 50000
Persona mount 2505 2409 5000
SpinButton mount 1970 1923 5000

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 21, 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 d8c377d:

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

@size-auditor
Copy link

size-auditor bot commented Apr 21, 2023

Asset size changes

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

Baseline commit: a59a1056a6110af48fd7a7ab189ced2a6a9567d1 (build)

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 21, 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.34 kB
57.141 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 a59a1056a6110af48fd7a7ab189ced2a6a9567d1

@fabricteam
Copy link
Collaborator

fabricteam commented Apr 21, 2023

🕵 fluentuiv9 No visual regressions between this PR and main

@tomi-msft tomi-msft marked this pull request as ready for review May 4, 2023 20:36
@tomi-msft tomi-msft requested a review from a team as a code owner May 4, 2023 20:36
@khmakoto khmakoto changed the title add pulse using scale fix: Adding pulse using scale May 4, 2023
@khmakoto khmakoto changed the title fix: Adding pulse using scale fix: Scale pulse animation with percentages and flip wave animation's direction May 4, 2023
@tomi-msft tomi-msft merged commit ff50792 into microsoft:master May 4, 2023
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request May 10, 2023
* master: (29 commits)
  feat(react-drawer): create DrawerBody component (microsoft#27581)
  feat(react-tree): TreeItem itemType restructure (microsoft#27799)
  feat: Implement state management for toasts (microsoft#27800)
  bugfix: fix VisibleFlatTreeItemGenerator omitting visible items (microsoft#27802)
  fix: overflowManager should always dispatch initial state (microsoft#27756)
  fix(react-badge): Remove white border around presence badge when on a dark background (microsoft#27780)
  react-tags: rename dismissable to dismissible (microsoft#27798)
  chore: update ownership of react-tags (microsoft#27795)
  applying package updates
  feat(react-tags): Replace `avatar` prop with `media`; polish styles for spacing (microsoft#27725)
  RFC appearance migration (microsoft#24181)
  chore(react-skeleton): Release react-skeleton to stable (microsoft#27767)
  fix(Coachmark): Remove positioning regression and update bounds on resize (microsoft#27782)
  applying package updates
  fix(v8): explicitly publish dist folder after node 16 upgrade (microsoft#27769)
  applying package updates
  fix: do not use outlineStyle: none in createCustomFocusIndicatorStyle (microsoft#26089)
  fix: Scale pulse animation with percentages and flip wave animation's direction (microsoft#27654)
  Publish dist folder that got removed due to node 16 upgrade (microsoft#27764)
  fix(scripts-update-release-notes): properly handle git for-each-ref cmd call to not fail release notes update (microsoft#27757)
  ...
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