-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Open
Labels
[Package] Components/packages/components/packages/components[Type] OverviewComprehensive, high level view of an area of focus often with multiple tracking issuesComprehensive, high level view of an area of focus often with multiple tracking issues
Description
After #40740, there are two different props in the Popover
component to affect is position: position
(legacy) and placement
(new)
In order to deprecate the legacy position
prop from the Popover
component, we should first migrate all current usages to the new placement
prop:
- Block switcher preview Popover: use new placement prop instead of legacy position prop #44387
- LinkControl: use Popover new placement prop instead of legacy position prop in README #44388
- Format Toolbar Popover: use new placement prop instead of legacy position prop #44389
- URL Input Popover: use new placement prop instead of legacy position prop #44390
- Button block Popover: use placement prop instead of legacy position prop #44392
- Navigation Link Popover: use placement prop instead of legacy position prop #44394
- Navigation Submenu Popover: use placement prop instead of legacy position prop #44395
- Autocompleter Popover: use placement prop instead of legacy position prop #44396
- Image format popover: use placement prop instead of legacy position prop #44398
- Link format Popover: use placement prop instead of legacy position prop #44399
- remove
position
fromPopover
's storybook - consider exporting the position-to-placement converstion utility for URLPopover, Dropdown, Tooltip and DotTip
- URLPopover: use new placement prop instead of legacy position prop #44391
- Dropdown (requires deprecating
position
in favour ofpopoverProps.placement
,Dropdown
: theposition
prop is made redundant bypopoverProps
#46858)Dropdown
: deprecateposition
prop, usepopoverProps
instead #46865 - Tooltip (requires deprecating
position
in favour ofplacement
too) Tooltip: addplacement
prop to replace deprecatedposition
#54264 - Components: add
placement
prop to replaceposition
inRangeControl
#70326 - Warning: Replace popoverProps.position to popoverProps.placement #70347
- Social Link: Use
placement
prop for popover positioning #70348 - Media List: Use 'placement' prop for popover positioning #70350
- Circular Option Picker: Use 'placement' prop for popover positioning #70359
- Any leftover
position={
orposition:
props:-
tooltipPosition
prop (ie.Button
component)
-
Metadata
Metadata
Assignees
Labels
[Package] Components/packages/components/packages/components[Type] OverviewComprehensive, high level view of an area of focus often with multiple tracking issuesComprehensive, high level view of an area of focus often with multiple tracking issues
Type
Projects
Status
In progress (owned) ⏳