Skip to content

chore: improves internal headless signature#28651

Merged
bsunderhus merged 1 commit intomicrosoft:masterfrom
bsunderhus:react-tree/chore--improves-headless-tree-signature
Jul 28, 2023
Merged

chore: improves internal headless signature#28651
bsunderhus merged 1 commit intomicrosoft:masterfrom
bsunderhus:react-tree/chore--improves-headless-tree-signature

Conversation

@bsunderhus
Copy link
Contributor

@bsunderhus bsunderhus commented Jul 26, 2023

Previous Behavior

New Behavior

  1. improves internal headless signature by exposing .add method
  2. renames internal parts from flat to headless
  3. makes the default value on tree item context to be virtualTreeRootId
  4. removes console warn for no tabbable element (since the tree can be empty...)

Related Issue(s)

  • Fixes #

@bsunderhus bsunderhus self-assigned this Jul 26, 2023
@github-actions github-actions bot added this to the July Project Cycle Q3 2023 milestone Jul 26, 2023
@bsunderhus bsunderhus marked this pull request as ready for review July 26, 2023 13:35
@bsunderhus bsunderhus requested a review from a team as a code owner July 26, 2023 13:35
@fabricteam
Copy link
Collaborator

Perf Analysis (@fluentui/react-components)

Scenario Render type Master Ticks PR Ticks Iterations Status
InfoButton mount 12 15 5000 Possible regression
All results

Scenario Render type Master Ticks PR Ticks Iterations Status
Avatar mount 587 601 5000
Button mount 280 300 5000
Field mount 1069 1042 5000
FluentProvider mount 653 640 5000
FluentProviderWithTheme mount 70 75 10
FluentProviderWithTheme virtual-rerender 65 70 10
FluentProviderWithTheme virtual-rerender-with-unmount 68 67 10
InfoButton mount 12 15 5000 Possible regression
MakeStyles mount 859 857 50000
Persona mount 1624 1611 5000
SpinButton mount 1295 1327 5000

@fabricteam
Copy link
Collaborator

📊 Bundle size report

Unchanged fixtures
Package & Exports Size (minified/GZIP)
react-components
react-components: Button, FluentProvider & webLightTheme
67.576 kB
18.225 kB
react-components
react-components: Accordion, Button, FluentProvider, Image, Menu, Popover
207.472 kB
57.814 kB
react-components
react-components: FluentProvider & webLightTheme
36.409 kB
12.003 kB
react-portal-compat
PortalCompatProvider
6.48 kB
2.203 kB
🤖 This report was generated against 0e9c438b78e58f346f61b15224c7b0020e4fe73e

@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 08bd72e:

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

@size-auditor
Copy link

size-auditor bot commented Jul 26, 2023

Asset size changes

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

Baseline commit: 0e9c438b78e58f346f61b15224c7b0020e4fe73e (build)

@bsunderhus bsunderhus merged commit 2f795af into microsoft:master Jul 28, 2023
@bsunderhus bsunderhus deleted the react-tree/chore--improves-headless-tree-signature branch July 28, 2023 07:21
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Aug 1, 2023
* master: (37 commits)
  release (microsoft#28696)
  Fixing re-render issue for all charts when empty (microsoft#28321)
  feat(FluentProvider): emit errors on duplicate IDs (microsoft#28670)
  applying package updates
  fix(react-positioning): autoSize causing position update to reach maximum (microsoft#28689)
  fix(react-tags-preview): fix InteractionTag hover styles (microsoft#28686)
  Accordion: export AccordionHeaderProvider (microsoft#28542)
  feat(react-shared-contexts): add AnnounceContext (microsoft#28654)
  Added VR tests for Breadcrumb (microsoft#28653)
  fix(react-menu): use outline for menuItem focus ring (microsoft#28685)
  [Bug]: Tree, vertical spacing of branches and children is inconsistent (microsoft#28681)
  feaTt(react-tree): adds openItems and checkedItems to tree callback data (microsoft#28669)
  applying package updates
  Add mountNode prop to combos (microsoft#28661)
  react-tags-preview: add more vr test (microsoft#28582)
  chore: migrate to nx 16.1.4 (microsoft#28583)
  applying package updates
  chore: improves internal headless signature (microsoft#28651)
  fix: remove margin from icon when ToolbarButton is vertical (microsoft#28658)
  applying package updates
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants