Skip to content

Use block naming for marking blocks as overridable in patterns #59268

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
youknowriad merged 20 commits into from
Mar 1, 2024
Merged

Use block naming for marking blocks as overridable in patterns #59268

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
ba91d58
Try using block metadata names instead of nano ids for pattern overri…
talldan Feb 22, 2024
92fb5db
Switch to using a const
talldan Feb 22, 2024
6d4b66e
Remove `values` property, reverting back to attributes being at root …
talldan Feb 26, 2024
ef59150
Add a migration to the new format
talldan Feb 28, 2024
fa3bb13
Add backwards compatibility
talldan Feb 28, 2024
3b45710
Use a map of clientIds to ids for back compat
talldan Feb 29, 2024
9a48af8
Fix bindings not being applied when using the rename option in List View
talldan Feb 29, 2024
4cbb6c5
Rename to addBindings to complement removeBindings
talldan Feb 29, 2024
8c7656a
Rename partial-syncing-controls file
talldan Feb 29, 2024
a2cf150
Rename pattern-partial-syncing hooks file to pattern-overrides
talldan Feb 29, 2024
42048f9
Remove nanoid dependency
talldan Feb 29, 2024
5e7c650
Update doc string
talldan Feb 29, 2024
1642af4
Fix linting issues
aaronrobertshaw Mar 1, 2024
4a415c1
More linting fixes
aaronrobertshaw Mar 1, 2024
bd5f4b4
Linting again
aaronrobertshaw Mar 1, 2024
035d0d6
Update e2e tests
talldan Mar 1, 2024
121ff2d
Fix reset button on individual blocks
talldan Mar 1, 2024
2f68377
Address feedback
talldan Mar 1, 2024
72299f0
Add fixtures for deprecations
talldan Mar 1, 2024
c0440bd
Use fill instead of click & type
talldan Mar 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Update doc string
  • Loading branch information
@talldan
talldan committed Feb 29, 2024
commit 5e7c6503f736faaab56f2617859d4773ebff2e55
6 changes: 3 additions & 3 deletions packages/patterns/src/components/use-set-pattern-bindings.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ export default function useSetPatternBindings(

useEffect( () => {
// Bindings should only be created when editing a wp_block post type,
// and also when there's a change to the user given name for the block.
// and also when there's a change to the user-given name for the block.
if (
currentPostType !== 'wp_block' ||
metadataName === prevMetadataName
Expand All @@ -70,7 +70,7 @@ export default function useSetPatternBindings(
return;
}

// The user given name for the block was deleted, remove the bindings.
// The user-given name for the block was deleted, remove the bindings.
if ( ! metadataName?.length && prevMetadataName?.length ) {
const updatedBindings = removeBindings(
bindings,
Expand All @@ -84,7 +84,7 @@ export default function useSetPatternBindings(
} );
}

// The user given name for the block was set, set the bindings.
// The user-given name for the block was set, set the bindings.
if ( ! prevMetadataName?.length && metadataName.length ) {
const updatedBindings = addBindings( bindings, syncedAttributes );
setAttributes( {
Expand Down