-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
feat(react-tree): Actions positioning and behaviour #26113
Conversation
📊 Bundle size report🤖 This report was generated against 96c5a1cbb36d728bebfbf05203d372a82b1968d8 |
Asset size changesSize Auditor did not detect a change in bundle size for any component! Baseline commit: 96c5a1cbb36d728bebfbf05203d372a82b1968d8 (build) |
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:
|
packages/react-components/react-tree/src/components/TreeItem/renderTreeItem.tsx
Show resolved
Hide resolved
packages/react-components/react-tree/src/components/BaseTreeItem/useBaseTreeItem.ts
Show resolved
Hide resolved
packages/react-components/react-tree/src/components/TreeItem/useTreeItem.tsx
Outdated
Show resolved
Hide resolved
packages/react-components/react-tree/src/components/TreeItem/useTreeItem.tsx
Outdated
Show resolved
Hide resolved
4f1bdc2
to
5a33f2e
Compare
5a33f2e
to
19ba7a2
Compare
packages/react-components/react-tree/src/components/TreeItem/useTreeItem.tsx
Outdated
Show resolved
Hide resolved
a69678b
to
8dda994
Compare
* 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) ...
* 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]>
* 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]>
New Behavior
ARIAButton
as the main root type for theTreeItem
Space
as an action for theTreeItem
in consequence of not usingARIAButton
actions
slotsactions
slots through grouppersMenu
and other portals onactions
slots