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

docs(react-drawer): improve drawer stories examples #28283

Conversation

marcosmoura
Copy link
Contributor

@marcosmoura marcosmoura commented Jun 21, 2023

  • Add missing example of Drawer overlay without backdrop (modalType="non-modal")
  • Overall improvements of stories to make them have same layout

@marcosmoura marcosmoura added Type: Documentation Component: Drawer The Fluent v9 Drawer component labels Jun 21, 2023
@marcosmoura marcosmoura self-assigned this Jun 21, 2023
@marcosmoura marcosmoura requested a review from a team as a code owner June 21, 2023 17:20
@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 602 595 5000
Button mount 291 295 5000
Field mount 1045 1061 5000
FluentProvider mount 652 637 5000
FluentProviderWithTheme mount 76 83 10
FluentProviderWithTheme virtual-rerender 69 65 10
FluentProviderWithTheme virtual-rerender-with-unmount 72 67 10
InfoButton mount 10 15 5000
MakeStyles mount 836 827 50000
Persona mount 1652 1545 5000
SpinButton mount 1309 1293 5000

@size-auditor
Copy link

size-auditor bot commented Jun 21, 2023

Asset size changes

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

Baseline commit: 1a34d5cde97318e5af25d7b9951a76625ee3d94b (build)

@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 0afcdef:

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-components
react-components: Button, FluentProvider & webLightTheme
65.162 kB
17.952 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
210.365 kB
58.697 kB
react-components
react-components: FluentProvider & webLightTheme
36.395 kB
11.996 kB
react-portal-compat
PortalCompatProvider
6.473 kB
2.196 kB
🤖 This report was generated against 1a34d5cde97318e5af25d7b9951a76625ee3d94b

@marcosmoura marcosmoura merged commit 3a08754 into microsoft:master Jun 22, 2023
@marcosmoura marcosmoura deleted the docs/react-drawer-stories-improvement branch June 22, 2023 11:05
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 22, 2023
* master:
  docs(react-drawer): improve drawer stories examples (microsoft#28283)
  bugfix: adds grid-template-columns to DialogBody styles to ensure grid template layout (microsoft#28272)
  Doc: Tree Infinite Scrolling (microsoft#28197)
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 22, 2023
* master: (95 commits)
  docs(react-drawer): improve drawer stories examples (microsoft#28283)
  bugfix: adds grid-template-columns to DialogBody styles to ensure grid template layout (microsoft#28272)
  Doc: Tree Infinite Scrolling (microsoft#28197)
  fix(react-card): infer a11y id from immediate header element (microsoft#28266)
  Fixed bugs and added more stories to the Breadcrumb (microsoft#28267)
  refactor: Keep vanillajs code only where needed (microsoft#28278)
  fix: correcting focus behavior of react-search (microsoft#28241)
  Tooltip : updated tooltip styles (microsoft#28264)
  applying package updates
  feat(react-tags): add styles for size (microsoft#28229)
  docs(react-dialog): update DialogTriggerOutsideDialog to include focus behavior (microsoft#28176)
  bugfix: Ensures dialog actions stretches on breakpoints (microsoft#28258)
  applying package updates
  fix: TableHeaderCell should not render button when not sortable (microsoft#28097)
  fix(react-file-type-icons): Map mhtml extension to html icon (microsoft#28112)
  Fix overlapping axis labels on smaller viewports (microsoft#28239)
  useArrowNavigationGroup grid-linear axis (microsoft#28253)
  applying package updates
  fix: Alert example missing aria-label for icon (microsoft#28234)
  Overflow divider fix (microsoft#28011)
  ...
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jun 22, 2023
* master:
  docs(react-drawer): best practices (microsoft#28040)
  docs(react-drawer): add missing documentation for Drawer stories (microsoft#28284)
  docs(react-drawer): improve type descriptions and fix TS circular references (microsoft#28282)
  docs(react-drawer): improve drawer stories examples (microsoft#28283)
  bugfix: adds grid-template-columns to DialogBody styles to ensure grid template layout (microsoft#28272)
  Doc: Tree Infinite Scrolling (microsoft#28197)
  fix(react-card): infer a11y id from immediate header element (microsoft#28266)
  Fixed bugs and added more stories to the Breadcrumb (microsoft#28267)
  refactor: Keep vanillajs code only where needed (microsoft#28278)
  fix: correcting focus behavior of react-search (microsoft#28241)
  Tooltip : updated tooltip styles (microsoft#28264)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Drawer The Fluent v9 Drawer component Type: Documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants