Skip to content

Commit

Permalink
Popover: lock the __experimentalPopoverPositionToPlacement function (#…
Browse files Browse the repository at this point in the history
…47505)

* Rename and stabilize __experimentalPopoverPositionToPlacement

* Rename internal tests import statement

* Revert "Rename internal tests import statement"

This reverts commit 4a0602c.

* Revert "Rename and stabilize __experimentalPopoverPositionToPlacement"

This reverts commit 0d0a7aa.

* Use lock/unlock APIs to lock the `positionToPlacement` function

* Update CHANGELOG

* Add __experimental prefix

* Update CHANGELOG
  • Loading branch information
ciampo authored and ntsekouras committed Feb 13, 2023
1 parent 4d61bcd commit 0d58119
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 3 deletions.
10 changes: 8 additions & 2 deletions packages/block-editor/src/components/url-popover/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { useState } from '@wordpress/element';
import {
Button,
Popover,
__experimentalPopoverPositionToPlacement as positionToPlacement,
experiments as componentsExperiments,
} from '@wordpress/components';
import { chevronDown } from '@wordpress/icons';
import deprecated from '@wordpress/deprecated';
Expand All @@ -16,6 +16,11 @@ import deprecated from '@wordpress/deprecated';
*/
import LinkViewer from './link-viewer';
import LinkEditor from './link-editor';
import { unlock } from '../../lock-unlock';

const { __experimentalPopoverLegacyPositionToPlacement } = unlock(
componentsExperiments
);

const DEFAULT_PLACEMENT = 'bottom';

Expand Down Expand Up @@ -46,7 +51,8 @@ function URLPopover( {
if ( placement !== undefined ) {
computedPlacement = placement;
} else if ( position !== undefined ) {
computedPlacement = positionToPlacement( position );
computedPlacement =
__experimentalPopoverLegacyPositionToPlacement( position );
}
computedPlacement = computedPlacement || DEFAULT_PLACEMENT;

Expand Down
2 changes: 2 additions & 0 deletions packages/components/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@
- `NavigatorButton`: Reuse `Button` types ([47754](https://github.com/WordPress/gutenberg/pull/47754)).
- `ToolsPanel`: Allow display of optional items when values are updated externally to item controls ([47727](https://github.com/WordPress/gutenberg/pull/47727)).
- `ToolsPanel`: Ensure display of optional items when values are updated externally and multiple blocks selected ([47864](https://github.com/WordPress/gutenberg/pull/47864)).
- `CustomSelectControl`: lock the `__experimentalShowSelectedHint` prop ([#47229](https://github.com/WordPress/gutenberg/pull/47229)).
- Lock the `__experimentalPopoverPositionToPlacement` function and rename it to `__experimentalPopoverLegacyPositionToPlacement` ([#47505](https://github.com/WordPress/gutenberg/pull/47505)).

## 23.3.0 (2023-02-01)

Expand Down
1 change: 0 additions & 1 deletion packages/components/src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,6 @@ export { default as PanelHeader } from './panel/header';
export { default as PanelRow } from './panel/row';
export { default as Placeholder } from './placeholder';
export { default as Popover } from './popover';
export { positionToPlacement as __experimentalPopoverPositionToPlacement } from './popover/utils';
export { default as QueryControls } from './query-controls';
export { default as __experimentalRadio } from './radio-group/radio';
export { default as __experimentalRadioGroup } from './radio-group';
Expand Down
2 changes: 2 additions & 0 deletions packages/components/src/private-apis.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ import { __dangerousOptInToUnstableAPIsOnlyForCoreModules } from '@wordpress/pri
* Internal dependencies
*/
import { default as CustomSelectControl } from './custom-select-control';
import { positionToPlacement as __experimentalPopoverLegacyPositionToPlacement } from './popover/utils';

export const { lock, unlock } =
__dangerousOptInToUnstableAPIsOnlyForCoreModules(
Expand All @@ -17,4 +18,5 @@ export const { lock, unlock } =
export const experiments = {};
lock( experiments, {
CustomSelectControl,
__experimentalPopoverLegacyPositionToPlacement,
} );

0 comments on commit 0d58119

Please sign in to comment.