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

feat(react-tree): Actions positioning and behaviour #26113

Conversation

bsunderhus
Copy link
Contributor

@bsunderhus bsunderhus commented Dec 27, 2022

New Behavior

  1. stops using ARIAButton as the main root type for the TreeItem
  2. stops supporting Space as an action for the TreeItem in consequence of not using ARIAButton
  3. Supports proper positioning of actions slots
  4. Supports proper navigation of actions slots through grouppers
  5. Supports usage of Menu and other portals on actions slots

@bsunderhus bsunderhus self-assigned this Dec 27, 2022
@fabricteam
Copy link
Collaborator

fabricteam commented Dec 27, 2022

📊 Bundle size report

🤖 This report was generated against 96c5a1cbb36d728bebfbf05203d372a82b1968d8

@size-auditor
Copy link

size-auditor bot commented Dec 27, 2022

Asset size changes

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

Baseline commit: 96c5a1cbb36d728bebfbf05203d372a82b1968d8 (build)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 27, 2022

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 8dda994:

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

@bsunderhus bsunderhus marked this pull request as ready for review December 28, 2022 10:20
@bsunderhus bsunderhus requested a review from a team as a code owner December 28, 2022 10:20
@bsunderhus bsunderhus force-pushed the react-tree/feat--actions-positioning-and-behavior branch from 4f1bdc2 to 5a33f2e Compare January 4, 2023 18:25
@bsunderhus bsunderhus requested a review from a team as a code owner January 4, 2023 18:25
@bsunderhus bsunderhus force-pushed the react-tree/feat--actions-positioning-and-behavior branch from 5a33f2e to 19ba7a2 Compare January 4, 2023 18:29
@bsunderhus bsunderhus requested a review from ling1726 January 6, 2023 15:34
@bsunderhus bsunderhus force-pushed the react-tree/feat--actions-positioning-and-behavior branch from a69678b to 8dda994 Compare January 9, 2023 12:02
@bsunderhus bsunderhus enabled auto-merge (squash) January 9, 2023 12:11
@bsunderhus bsunderhus merged commit ab386c3 into microsoft:master Jan 9, 2023
@bsunderhus bsunderhus deleted the react-tree/feat--actions-positioning-and-behavior branch January 9, 2023 12:41
marcosmoura added a commit to marcosmoura/fluentui that referenced this pull request Jan 10, 2023
* master: (29 commits)
  applying package updates
  fix: web component menu layering bug (microsoft#26260)
  Azure Theme: reduced spin button height to match TextField / DropDowns at 24px height (microsoft#26265)
  Update styling for contentBefore and contentAfter input slots (microsoft#26115)
  chore: Update Switch to use griffel reset styles (microsoft#26007)
  Fix: Allow root slot refs to merge with focus refs in Slider (microsoft#26243)
  applying package updates
  revert: MenuItem root slot only supports div (microsoft#26261)
  perf: Don't render Checkbox icon when unchecked (microsoft#26248)
  fix: Select disabled state hover style, Combobox disabled state open on chevron click (microsoft#26068)
  applying package updates
  chore: add more temporary codeowner rools for tooling config files (microsoft#26255)
  fix: stops using ARIAButton types for MenuItem root (microsoft#26257)
  refactor: Cleanup unused code (microsoft#26219)
  fix: MenuList props should win over context props (microsoft#26252)
  feat(react-tree):  Actions positioning and behaviour (microsoft#26113)
  BREAKING(TableCellLayout): `wrapper` slot renamed to `content` (microsoft#26220)
  fix(scripts): make lint errors reporting propagate to STDOUT during pre-commit (lint-staged exec) (microsoft#26212)
  fix: Minimum visible overflow items should be respected (microsoft#26194)
  docs: Fix typos in react-table docs (microsoft#26213)
  ...
q1b pushed a commit to q1b/fluentui that referenced this pull request Jan 24, 2023
* feat(react-tree): properly position actions

* chore: updates API

* chore: fix tests and groupper className

* Update packages/react-components/react-tree/src/components/TreeItem/useTreeItem.tsx

Co-authored-by: ling1726 <[email protected]>

* chore: ensures functionality with Menu

* chore: adds role presentation to groupper

Co-authored-by: ling1726 <[email protected]>
Hotell pushed a commit to Hotell/fluentui that referenced this pull request Feb 9, 2023
* feat(react-tree): properly position actions

* chore: updates API

* chore: fix tests and groupper className

* Update packages/react-components/react-tree/src/components/TreeItem/useTreeItem.tsx

Co-authored-by: ling1726 <[email protected]>

* chore: ensures functionality with Menu

* chore: adds role presentation to groupper

Co-authored-by: ling1726 <[email protected]>
@bsunderhus bsunderhus mentioned this pull request Mar 13, 2023
22 tasks
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