diff --git a/packages/block-editor/src/components/block-list/index.native.js b/packages/block-editor/src/components/block-list/index.native.js index 05c2898899ca3..b925a724809ec 100644 --- a/packages/block-editor/src/components/block-list/index.native.js +++ b/packages/block-editor/src/components/block-list/index.native.js @@ -96,9 +96,6 @@ export class BlockList extends Component { ListHeaderComponent={ header } ListEmptyComponent={ this.renderDefaultBlockAppender } ListFooterComponent={ withFooter && this.renderBlockListFooter } - getItemLayout={ ( data, index ) => { - return { length: 0, offset: 0, index }; - } } /> { renderAppender && blockClientIds.length > 0 && diff --git a/packages/block-editor/src/components/inner-blocks/index.native.js b/packages/block-editor/src/components/inner-blocks/index.native.js index 2b6a6c9320c92..6d2d4290772fa 100644 --- a/packages/block-editor/src/components/inner-blocks/index.native.js +++ b/packages/block-editor/src/components/inner-blocks/index.native.js @@ -122,6 +122,7 @@ class InnerBlocks extends Component { rootClientId={ clientId } renderAppender={ renderAppender } withFooter={ false } + isFullyBordered={ true } /> ) } diff --git a/packages/block-editor/src/components/inserter/menu.native.js b/packages/block-editor/src/components/inserter/menu.native.js index a03e5f818540c..591932dde8ab6 100644 --- a/packages/block-editor/src/components/inserter/menu.native.js +++ b/packages/block-editor/src/components/inserter/menu.native.js @@ -21,6 +21,15 @@ import { BottomSheet, Icon } from '@wordpress/components'; import styles from './style.scss'; export class InserterMenu extends Component { + constructor() { + super( ...arguments ); + + this.onLayout = this.onLayout.bind( this ); + this.state = { + numberOfColumns: this.calculateNumberOfColumns(), + }; + } + componentDidMount() { this.onOpen(); } @@ -47,9 +56,13 @@ export class InserterMenu extends Component { this.props.hideInsertionPoint(); } + onLayout() { + const numberOfColumns = this.calculateNumberOfColumns(); + this.setState( { numberOfColumns } ); + } + render() { const { getStylesFromColorScheme } = this.props; - const numberOfColumns = this.calculateNumberOfColumns(); const bottomPadding = styles.contentBottomPadding; const modalIconWrapperStyle = getStylesFromColorScheme( styles.modalIconWrapper, styles.modalIconWrapperDark ); const modalIconStyle = getStylesFromColorScheme( styles.modalIcon, styles.modalIconDark ); @@ -63,10 +76,11 @@ export class InserterMenu extends Component { hideHeader > diff --git a/packages/block-library/src/media-text/edit.native.js b/packages/block-library/src/media-text/edit.native.js index 73532a322916a..f83092d287431 100644 --- a/packages/block-library/src/media-text/edit.native.js +++ b/packages/block-library/src/media-text/edit.native.js @@ -10,7 +10,7 @@ import { View } from 'react-native'; import { __, _x } from '@wordpress/i18n'; import { BlockControls, - BlockVerticalAlignmentToolbar, + // BlockVerticalAlignmentToolbar, InnerBlocks, withColors, } from '@wordpress/block-editor'; @@ -131,15 +131,16 @@ class MediaTextEdit extends Component { attributes, backgroundColor, setAttributes, - isMobile, + // isMobile, } = this.props; const { - isStackedOnMobile, + // isStackedOnMobile, mediaPosition, mediaWidth, verticalAlignment, } = attributes; - const shouldStack = isStackedOnMobile && isMobile; + const shouldStack = false; // We are temporarily not stacking until we fix alignment buttons + // const shouldStack = isStackedOnMobile && isMobile; // <<< Original line const temporaryMediaWidth = shouldStack ? 100 : ( this.state.mediaWidth || mediaWidth ); const widthString = `${ temporaryMediaWidth }%`; const containerStyles = { @@ -165,9 +166,9 @@ class MediaTextEdit extends Component { onClick: () => setAttributes( { mediaPosition: 'right' } ), } ]; - const onVerticalAlignmentChange = ( alignment ) => { + /* const onVerticalAlignmentChange = ( alignment ) => { setAttributes( { verticalAlignment: alignment } ); - }; + }; */ return ( <> @@ -175,11 +176,12 @@ class MediaTextEdit extends Component { + { /* // Temporarily commenting out until alignment functionality is fixed + /> */ } diff --git a/packages/block-library/src/media-text/media-container.native.js b/packages/block-library/src/media-text/media-container.native.js index 664cf43e1e5dc..84d2c6dc836bd 100644 --- a/packages/block-library/src/media-text/media-container.native.js +++ b/packages/block-library/src/media-text/media-container.native.js @@ -62,17 +62,14 @@ class MediaContainer extends Component { } componentDidMount() { - const { mediaId, mediaUrl, onSelectMedia } = this.props; + const { mediaId, mediaUrl, onMediaUpdate, mediaType } = this.props; if ( mediaId && mediaUrl && ! isURL( mediaUrl ) ) { - if ( mediaUrl.indexOf( 'file:' ) === 0 ) { - requestMediaImport( mediaUrl, ( id, url, type ) => { + if ( mediaUrl.indexOf( 'file:' ) === 0 && mediaType === MEDIA_TYPE_IMAGE ) { + // We don't want to call this for video because it is starting a media upload for the cover url + requestMediaImport( mediaUrl, ( id, url ) => { if ( url ) { - onSelectMedia( { - media_type: type, - id, - url, - } ); + onMediaUpdate( { id, url } ); } } ); }