Skip to content

Conversation

@ling1726
Copy link
Contributor

@ling1726 ling1726 commented Sep 29, 2023

Implenents animations for the MessageBar. There are two possible cases:

  • Animation on exit only - in the event of a page refresh, message bars should not animate on enter
  • Animation on enter and exit - in the event of new message bars appearing during runtime, mesage bars should animate on enter.

Animation is handled for collections of MessageBar components through the MessageBarGroup component. The group component owns the animation styles and wraps MessageBar components with transitions so that the MessageBar can still be used stand alone without transitions

Addresses: #22579

@fabricteam
Copy link
Collaborator

fabricteam commented Sep 29, 2023

📊 Bundle size report

🤖 This report was generated against f61033d77d6852838c580bfa7379d026128c2dbb

@codesandbox-ci
Copy link

codesandbox-ci bot commented Sep 29, 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 7884909:

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

@ling1726 ling1726 force-pushed the react-message-bar/feat/motion branch 3 times, most recently from f6826fb to 56ae41c Compare September 29, 2023 16:38
@ling1726 ling1726 force-pushed the react-message-bar/feat/motion branch from bcce648 to 18b81d2 Compare October 2, 2023 12:58
@ling1726 ling1726 marked this pull request as ready for review October 2, 2023 12:58
@ling1726 ling1726 requested a review from a team as a code owner October 2, 2023 12:58
@size-auditor
Copy link

size-auditor bot commented Oct 2, 2023

Asset size changes

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

Baseline commit: f61033d77d6852838c580bfa7379d026128c2dbb (build)

@ling1726 ling1726 merged commit 20ce41b into microsoft:master Oct 2, 2023
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 4, 2023
* master: (146 commits)
  remove a11y test ux from theme designer (microsoft#29379)
  feat: preview release (microsoft#29377)
  Fixed divider in the tooltip (microsoft#29357)
  fix (microsoft#29376)
  docs: Initial documentation pass (microsoft#29372)
  feat: Integrate Announce context (microsoft#29362)
  chore(react-tags): stable release (microsoft#29355)
  chore: delegate focus outline to layout components (microsoft#29293)
  Vertical bar chart line bug (microsoft#29299)
  applying package updates
  Adding axis title examples for line, area, vertical bar and vertical stacked bar charts (microsoft#29254)
  A11y: Pagination: Add tooltip for all icon buttons  (microsoft#29155)
  A11y: Fix aria attribute for page number and add correct role (microsoft#29349)
  feat: Implement motion for MessageBar (microsoft#29339)
  feat: creates getIntrinsicElementProps to replace getNativeElementProps on slots creation (microsoft#29310)
  chore: replace makeStyles with makeResetStyles (microsoft#29338)
  applying package updates
  ci(github/policies): remove bot rules that created and added `Component/Package $1`: labels on PRs (microsoft#29301)
  fix: handle multiple classes in PortalCompatProvider (microsoft#29351)
  applying package updates
  ...
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Oct 4, 2023
* master: (49 commits)
  remove a11y test ux from theme designer (microsoft#29379)
  feat: preview release (microsoft#29377)
  Fixed divider in the tooltip (microsoft#29357)
  fix (microsoft#29376)
  docs: Initial documentation pass (microsoft#29372)
  feat: Integrate Announce context (microsoft#29362)
  chore(react-tags): stable release (microsoft#29355)
  chore: delegate focus outline to layout components (microsoft#29293)
  Vertical bar chart line bug (microsoft#29299)
  applying package updates
  Adding axis title examples for line, area, vertical bar and vertical stacked bar charts (microsoft#29254)
  A11y: Pagination: Add tooltip for all icon buttons  (microsoft#29155)
  A11y: Fix aria attribute for page number and add correct role (microsoft#29349)
  feat: Implement motion for MessageBar (microsoft#29339)
  feat: creates getIntrinsicElementProps to replace getNativeElementProps on slots creation (microsoft#29310)
  chore: replace makeStyles with makeResetStyles (microsoft#29338)
  applying package updates
  ci(github/policies): remove bot rules that created and added `Component/Package $1`: labels on PRs (microsoft#29301)
  fix: handle multiple classes in PortalCompatProvider (microsoft#29351)
  applying package updates
  ...
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.

3 participants