Don't use wordpress/warning package for displaying the warning becaus…

…e it's not going to work in the browser. It's intended to be used with Node.
WordPress · Oct 27, 2021 · 70ed0ed · 70ed0ed
1 parent 1dd6bbc
commit 70ed0ed
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 4 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/edit-navigation/package.json b/packages/edit-navigation/package.json
@@ -49,7 +49,6 @@
 		"@wordpress/media-utils": "file:../media-utils",
 		"@wordpress/notices": "file:../notices",
 		"@wordpress/url": "file:../url",
-		"@wordpress/warning": "file:../warning",
 		"classnames": "^2.3.1",
 		"lodash": "^4.17.21",
 		"rememo": "^3.0.0",

diff --git a/packages/edit-navigation/src/store/actions.js b/packages/edit-navigation/src/store/actions.js
@@ -9,7 +9,6 @@ import { difference, zip } from 'lodash';
 import { __, sprintf } from '@wordpress/i18n';
 import { store as noticesStore } from '@wordpress/notices';
 import { store as coreDataStore } from '@wordpress/core-data';
-import { default as logWarning } from '@wordpress/warning';
 
 /**
  * Internal dependencies
@@ -210,7 +209,7 @@ const batchUpdateMenuItems = ( navigationBlock, menuId ) => async ( {
 		.filter( ( { block } ) => ! isBlockSupportedInNav( block ) )
 		.map( ( { block } ) => block.name );
 	if ( unsupportedMenuItems.length ) {
-		logWarning(
+		window.console.warn(
 			sprintf(
 				// translators: %s: Name of block (i.e. core/legacy-widget)
 				__(