Skip to content

feat(react-drawer): use dialog component and implement size prop#27431

Merged
marcosmoura merged 40 commits into
microsoft:masterfrom
marcosmoura:marcosmoura/feat/drawer-component
Apr 13, 2023
Merged

feat(react-drawer): use dialog component and implement size prop#27431
marcosmoura merged 40 commits into
microsoft:masterfrom
marcosmoura:marcosmoura/feat/drawer-component

Conversation

@marcosmoura

@marcosmoura marcosmoura commented Apr 4, 2023

Copy link
Copy Markdown
Contributor

This component is not yet available.

Previous Behavior

Drawer was using plain div as its root slot.

New Behavior

  • Use Dialog component as the base for Drawer
  • Add support for setting size for the Drawer. It implements the sizing values described on the Design Specs.

* master: (47 commits)
  fix: Fix vertical alignment of inline Checkbox when checked vs. unchecked (microsoft#27324)
  applying package updates
  Fix: Revert microsoft#26931 change due to new bug it introduced (microsoft#27323)
  chore(react-infobutton): Renaming content slot to info (microsoft#27316)
  applying package updates
  Fix bad release mismatch of react-select (microsoft#27321)
  applying package updates
  Fix react-combobox mismatch after bad release (microsoft#27319)
  docs: add section about media query sorting (microsoft#27307)
  applying package updates
  fix: sync packages and changefiles after bad release (microsoft#27317)
  applying package updates
  chore(react-datepicker-compat): Cleanup types, unused code, and restore utils tests (microsoft#27297)
  [Fluent 2 Theme] Fixing link color in dark theme (microsoft#27203)
  Tree documentation and stories (microsoft#27270)
  feat: Render theme CSS variables in SSR style element (microsoft#27277)
  chore (scripts): update create-package plop templates to use swc/helpers in deps (microsoft#27286)
  fix(DialogContent): adds padding to allow focus visibility (microsoft#27248)
  Adding Card component style hooks to shared context (microsoft#27295)
  Virtualizer: Static scroll hooks and embedded scrollview (microsoft#26985)
  ...
@marcosmoura
marcosmoura requested review from a team as code owners April 4, 2023 13:33
@marcosmoura marcosmoura self-assigned this Apr 4, 2023
@marcosmoura marcosmoura added the Component: Drawer The Fluent v9 Drawer component label Apr 4, 2023
@marcosmoura marcosmoura changed the title feat(react-drawer): add size prop feat(react-drawer): use dialog component and implement size prop Apr 4, 2023
* master: (39 commits)
  Added merge tests for custom styles (microsoft#27086)
  fix: make Option's checkIcon slot render conditionally (microsoft#27409)
  docs: add subcomponents to Combobox and Dropdown docs (microsoft#27410)
  Breadcrumb spec (microsoft#26742)
  bugfix(react-utilities): increments SlotRenderFunction signature to include children (microsoft#27377)
  fix: Menu should not steal focus on close (microsoft#27414)
  Fixed Overflow component when there's more then one children element (microsoft#27402)
  feat(scripts-executors): support custom base and printing all affected packages for checkIfPackagesAffected (microsoft#27364)
  applying package updates
  feat(scripts-tasks): make perf test setup generic (microsoft#26920)
  fix: ColorPicker preview color updated to ignore forced colors (microsoft#27394)
  fix: Fix babel build error on windows (microsoft#27406)
  chore(react-datepicker-compat): Add vr-tests and bundle-size fixtures for DatePicker (microsoft#27351)
  fix(react-dialog): DialogContent scrollbar always visible (microsoft#27367)
  docs: Add tokens search (microsoft#27265)
  chore(scripts-tasks): remove not needed api-extractor task logs (microsoft#27094)
  chore(react-tree): adds e2e flat tree tests (microsoft#27318)
  applying package updates
  Accessibility fix for empty vertical bar chart (microsoft#26254)
  fix(react-datepicker-compat): Fix theming issues with all themes and WHCM (microsoft#27361)
  ...
@marcosmoura marcosmoura mentioned this pull request Apr 4, 2023
46 tasks
@fabricteam

fabricteam commented Apr 4, 2023

Copy link
Copy Markdown
Collaborator

📊 Bundle size report

🤖 This report was generated against e5c39d1494652e5803b6e8822a2e3fb894aec811

@codesandbox-ci

codesandbox-ci Bot commented Apr 4, 2023

Copy link
Copy Markdown

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 f46484c:

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

@size-auditor

size-auditor Bot commented Apr 4, 2023

Copy link
Copy Markdown

Asset size changes

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

Baseline commit: e5c39d1494652e5803b6e8822a2e3fb894aec811 (build)

Comment thread packages/react-components/react-drawer/src/components/Drawer/useDrawerStyles.ts Outdated
Comment thread packages/react-components/react-drawer/src/components/Drawer/useDrawer.tsx Outdated
Comment thread packages/react-components/react-drawer/src/components/Drawer/Drawer.types.ts Outdated
Comment thread packages/react-components/react-drawer/src/components/Drawer/renderDrawer.tsx Outdated
Comment thread packages/react-components/react-drawer/src/components/Drawer/renderDrawer.tsx Outdated
Comment thread packages/react-components/react-drawer/src/components/Drawer/Drawer.types.ts Outdated

@Hotell Hotell left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM,

make sure to update against latest master as you have out of sync deps

image

@marcosmoura
marcosmoura merged commit 57eef05 into microsoft:master Apr 13, 2023
@marcosmoura
marcosmoura deleted the marcosmoura/feat/drawer-component branch April 13, 2023 12:09
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants