Update: Removed editor store usage from native mobile block editor in…

…serter
WordPress · Nov 27, 2019 · b7331fe · b7331fe
1 parent 336f183
commit b7331fe
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 6 deletions.
diff --git a/packages/block-editor/src/components/inserter/menu.native.js b/packages/block-editor/src/components/inserter/menu.native.js
@@ -116,6 +116,7 @@ export default compose(
  getBlockName,
  getBlockRootClientId,
  getBlockSelectionEnd,
+ getSettings,
  } = select( 'core/block-editor' );
  const {
  getChildBlockNames,
@@ -130,10 +131,13 @@ export default compose(
  }
  const destinationRootBlockName = getBlockName( destinationRootClientId );
 
+ const { __experimentalShouldInsertAtTheTop: shouldInsertAtTheTop } = getSettings();
+
  return {
  rootChildBlocks: getChildBlockNames( destinationRootBlockName ),
  items: getInserterItems( destinationRootClientId ),
  destinationRootClientId,
+ shouldInsertAtTheTop,
  };
  } ),
  withDispatch( ( dispatch, ownProps, { select } ) => {
@@ -154,13 +158,10 @@ export default compose(
  getBlockSelectionEnd,
  getBlockOrder,
  } = select( 'core/block-editor' );
- const {
- isPostTitleSelected,
- } = select( 'core/editor' );
- const { clientId, destinationRootClientId, isAppender } = ownProps;
+ const { clientId, destinationRootClientId, isAppender, shouldInsertAtTheTop } = ownProps;
 
  // if post title is selected insert as first block
- if ( isPostTitleSelected() ) {
+ if ( shouldInsertAtTheTop ) {
  return 0;
  }
 

diff --git a/packages/editor/src/components/provider/index.js b/packages/editor/src/components/provider/index.js
@@ -78,7 +78,8 @@ class EditorProvider extends Component {
  settings,
  reusableBlocks,
  hasUploadPermissions,
- canUserUseUnfilteredHTML
+ canUserUseUnfilteredHTML,
+ __experimentalShouldInsertAtTheTop
  ) {
  return {
  ...pick( settings, [
@@ -113,6 +114,7 @@ class EditorProvider extends Component {
  __experimentalReusableBlocks: reusableBlocks,
  __experimentalFetchLinkSuggestions: fetchLinkSuggestions,
  __experimentalCanUserUseUnfilteredHTML: canUserUseUnfilteredHTML,
+ __experimentalShouldInsertAtTheTop,
  };
  }
 
@@ -158,6 +160,7 @@ class EditorProvider extends Component {
  reusableBlocks,
  resetEditorBlocksWithoutUndoLevel,
  hasUploadPermissions,
+ isPostTitleSelected,
  } = this.props;
 
  if ( ! isReady ) {
@@ -169,6 +172,7 @@ class EditorProvider extends Component {
  reusableBlocks,
  hasUploadPermissions,
  canUserUseUnfilteredHTML,
+ isPostTitleSelected,
  );
 
  return (
@@ -203,6 +207,7 @@ export default compose( [
  getEditorSelectionStart,
  getEditorSelectionEnd,
  __experimentalGetReusableBlocks,
+ isPostTitleSelected,
  } = select( 'core/editor' );
  const { canUser } = select( 'core' );
 
@@ -214,6 +219,8 @@ export default compose( [
  selectionEnd: getEditorSelectionEnd(),
  reusableBlocks: __experimentalGetReusableBlocks(),
  hasUploadPermissions: defaultTo( canUser( 'create', 'media' ), true ),
+ // This selector is only defined on mobile.
+ isPostTitleSelected: isPostTitleSelected && isPostTitleSelected(),
  };
  } ),
  withDispatch( ( dispatch ) => {