Skip to content

Commit

Permalink
fix (performance): revert memo, add timeout in useEffect (#3258)
Browse files Browse the repository at this point in the history
* Revert "fix (performance): fixes freezing when creating and adding synced patterns in WP 6.6 (#3245)"

This reverts commit 16947b5.

* add timeout in useEffect
  • Loading branch information
mxkae authored Jul 22, 2024
1 parent 35c9ddb commit bbf70f1
Show file tree
Hide file tree
Showing 45 changed files with 116 additions and 252 deletions.
8 changes: 2 additions & 6 deletions src/block-components/advanced/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,8 @@
* External dependencies
*/
import { BlockCss } from '~stackable/components'
import { memo } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -339,13 +338,10 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const Style = props => {
return <Styles { ...props } />
}

Style.Content = props => {
return <Styles.Content { ...props } />
return <Styles { ...props } />
}
17 changes: 6 additions & 11 deletions src/block-components/alignment/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@
*/
import { BlockCss } from '~stackable/components'

import { memo } from '@wordpress/element'

const _AlignmentStyles = props => {
const AlignmentStyles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -210,7 +208,7 @@ const _AlignmentStyles = props => {
styleRule="columnGap"
attrName="innerBlockColumnGap"
key="innerBlockColumnGapEdit"
format="%spx"
format={ `%spx` }
responsive="all"
enabledCallback={ getAttribute => getAttribute( 'innerBlockOrientation' ) === 'horizontal' }
dependencies={ [
Expand All @@ -225,7 +223,7 @@ const _AlignmentStyles = props => {
styleRule="columnGap"
attrName="innerBlockColumnGap"
key="innerBlockColumnGapSave"
format="%spx"
format={ `%spx` }
responsive="all"
enabledCallback={ getAttribute => getAttribute( 'innerBlockOrientation' ) === 'horizontal' }
dependencies={ [
Expand All @@ -240,7 +238,7 @@ const _AlignmentStyles = props => {
styleRule="rowGap"
attrName="innerBlockRowGap"
key="innerBlockRowGapEdit"
format="%spx"
format={ `%spx` }
responsive="all"
enabledCallback={ getAttribute => {
return getAttribute( 'innerBlockOrientation' ) !== 'horizontal' ||
Expand All @@ -259,7 +257,7 @@ const _AlignmentStyles = props => {
styleRule="rowGap"
attrName="innerBlockRowGap"
key="innerBlockRowGapSave"
format="%spx"
format={ `%spx` }
responsive="all"
enabledCallback={ getAttribute => {
return getAttribute( 'innerBlockOrientation' ) !== 'horizontal' ||
Expand Down Expand Up @@ -307,13 +305,10 @@ const _AlignmentStyles = props => {
)
}

const AlignmentStyles = memo( _AlignmentStyles )
AlignmentStyles.Content = _AlignmentStyles

export const Style = props => {
return <AlignmentStyles { ...props } />
}

Style.Content = props => {
return <AlignmentStyles.Content { ...props } />
return <AlignmentStyles { ...props } />
}
6 changes: 3 additions & 3 deletions src/block-components/block-div/style.js
Original file line number Diff line number Diff line change
@@ -1,13 +1,13 @@
/**
* Internal dependencies
*/
import { Fragment, memo } from '@wordpress/element'
import { Fragment } from '@wordpress/element'
import { useBlockAttributesContext } from '~stackable/hooks'
import {
BorderStyle, SizeStyle, BackgroundStyle,
} from '../helpers'

export const Style = memo( props => {
export const Style = props => {
const hasBackground = useBlockAttributesContext( attributes => attributes.hasBackground )

return (
Expand All @@ -17,7 +17,7 @@ export const Style = memo( props => {
<SizeStyle { ...props } attrNameTemplate="block%s" />
</>
)
} )
}

Style.Content = props => {
return (
Expand Down
9 changes: 2 additions & 7 deletions src/block-components/button/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,7 @@ import { BorderStyle } from '../helpers/borders'
import { Icon } from '../icon'
import { BlockCss } from '~stackable/components'

import { memo } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -192,9 +190,6 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const Style = props => {
const {
selector = '',
Expand Down Expand Up @@ -235,7 +230,7 @@ Style.Content = props => {

return (
<>
<Styles.Content
<Styles
{ ...props }
attrNameTemplate={ attrNameTemplate }
/>
Expand Down
8 changes: 2 additions & 6 deletions src/block-components/column/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,8 @@
*/
import { __getValue } from '~stackable/util'
import { BlockCss } from '~stackable/components'
import { memo } from '@wordpress/element'

const _ColumnStyles = props => {
const ColumnStyles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -103,13 +102,10 @@ const _ColumnStyles = props => {
)
}

const ColumnStyles = memo( _ColumnStyles )
ColumnStyles.Content = _ColumnStyles

export const Style = props => {
return <ColumnStyles { ...props } />
}

Style.Content = props => {
return <ColumnStyles.Content { ...props } />
return <ColumnStyles { ...props } />
}
9 changes: 3 additions & 6 deletions src/block-components/columns/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,9 @@ import { useBlockAttributesContext } from '~stackable/hooks'
/**
* WordPress dependencies
*/
import { Fragment, memo } from '@wordpress/element'
import { Fragment } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -186,9 +186,6 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const Style = props => {
const columnArrangement = useBlockAttributesContext( attributes => attributes.columnArrangementMobile || attributes.columnArrangementTablet )
const numColumns = ( columnArrangement || '' ).split( ',' ).length
Expand All @@ -197,5 +194,5 @@ export const Style = props => {

Style.Content = props => {
const numColumns = ( props.attributes.columnArrangementMobile || props.attributes.columnArrangementTablet || '' ).split( ',' ).length
return <Styles.Content { ...props } numColumns={ numColumns } />
return <Styles { ...props } numColumns={ numColumns } />
}
6 changes: 2 additions & 4 deletions src/block-components/container-div/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,9 +6,7 @@ import {
} from '../helpers'
import { useBlockAttributesContext } from '~stackable/hooks'

import { memo } from '@wordpress/element'

export const Style = memo( props => {
export const Style = props => {
const {
backgroundSelector = '.%s-container',
borderSelector = '.%s-container',
Expand Down Expand Up @@ -52,7 +50,7 @@ export const Style = memo( props => {
/>
</>
)
} )
}

Style.defaultProps = {
options: {},
Expand Down
5 changes: 2 additions & 3 deletions src/block-components/effects-animations/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,11 @@
* WordPress dependencies
*/
import { applyFilters } from '@wordpress/hooks'
import { memo } from '@wordpress/element'

export const Style = memo( props => {
export const Style = props => {
const EffectStyles = applyFilters( 'stackable.block-component.effects-animations.style', null )
return EffectStyles && <EffectStyles { ...props } />
} )
}

Style.Content = props => {
const EffectStyles = applyFilters( 'stackable.block-component.effects-animations.style.content', null )
Expand Down
9 changes: 2 additions & 7 deletions src/block-components/helpers/backgrounds/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@
*/
import { BlockCss } from '~stackable/components'

import { memo } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -314,13 +312,10 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const BackgroundStyle = props => {
return <Styles { ...props } />
}

BackgroundStyle.Content = props => {
return <Styles.Content { ...props } />
return <Styles { ...props } />
}
9 changes: 2 additions & 7 deletions src/block-components/helpers/borders/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@
*/
import { BlockCss } from '~stackable/components'

import { memo } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -229,13 +227,10 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const BorderStyle = props => {
return <Styles { ...props } />
}

BorderStyle.Content = props => {
return <Styles.Content { ...props } />
return <Styles { ...props } />
}
9 changes: 2 additions & 7 deletions src/block-components/helpers/flex-gap/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@
*/
import { BlockCss } from '~stackable/components'

import { memo } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -42,13 +40,10 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const FlexGapStyles = props => {
return <Styles { ...props } />
}

FlexGapStyles.Content = props => {
return <Styles.Content { ...props } />
return <Styles { ...props } />
}
9 changes: 2 additions & 7 deletions src/block-components/helpers/size/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,7 @@
*/
import { BlockCss } from '~stackable/components'

import { memo } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -271,13 +269,10 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const SizeStyle = props => {
return <Styles { ...props } />
}

SizeStyle.Content = props => {
return <Styles.Content { ...props } />
return <Styles { ...props } />
}
10 changes: 3 additions & 7 deletions src/block-components/icon/style.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,9 +7,8 @@ import { BlockCss } from '~stackable/components'
* WordPress dependencies
*/
import { applyFilters } from '@wordpress/hooks'
import { memo } from '@wordpress/element'

const _Styles = props => {
const Styles = props => {
const propsToPass = {
...props,
version: props.version,
Expand Down Expand Up @@ -305,9 +304,6 @@ const _Styles = props => {
)
}

const Styles = memo( _Styles )
Styles.Content = _Styles

export const Style = props => {
const IndivIconStyles = applyFilters( 'stackable.block-component.icon.indiv-icon-style', null )

Expand All @@ -318,10 +314,10 @@ export const Style = props => {
}

Style.Content = props => {
const IndivIconStyles = applyFilters( 'stackable.block-component.icon.indiv-icon-style.content', null )
const IndivIconStyles = applyFilters( 'stackable.block-component.icon.indiv-icon-style', null )

return <>
<Styles.Content { ...props } />
<Styles { ...props } />
{ IndivIconStyles && <IndivIconStyles { ...props } /> }
</>
}
Loading

0 comments on commit bbf70f1

Please sign in to comment.