Skip to content

Support reusable blocks for multi-selection #9732

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 6 commits into from
Sep 12, 2018
Merged

Support reusable blocks for multi-selection #9732

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
d4d9758
Support converting a multiselection to a reusable block
youknowriad Sep 10, 2018
42791c9
Add e2e test for multi-selection reusable blocks
youknowriad Sep 10, 2018
10363e2
Remove awareness of `core/template` server-side
youknowriad Sep 10, 2018
66a33d8
Fix unit tests
youknowriad Sep 10, 2018
23a9e98
Update the template block icon
youknowriad Sep 12, 2018
aae9f43
Tweaks per review
youknowriad Sep 12, 2018
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
Tweaks per review
  • Loading branch information
@youknowriad
youknowriad committed Sep 12, 2018
commit aae9f4333f2ff88030913c30ba78daae726ef7e2
4 changes: 2 additions & 2 deletions block-library/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,12 +36,12 @@ import * as nextpage from '../packages/block-library/src/nextpage';
import * as preformatted from '../packages/block-library/src/preformatted';
import * as pullquote from '../packages/block-library/src/pullquote';
import * as reusableBlock from '../packages/block-library/src/block';
import * as reusableTemplate from '../packages/block-library/src/template';
import * as separator from '../packages/block-library/src/separator';
import * as shortcode from '../packages/block-library/src/shortcode';
import * as spacer from '../packages/block-library/src/spacer';
import * as subhead from '../packages/block-library/src/subhead';
import * as table from '../packages/block-library/src/table';
import * as template from '../packages/block-library/src/template';
import * as textColumns from '../packages/block-library/src/text-columns';
import * as verse from '../packages/block-library/src/verse';
import * as video from '../packages/block-library/src/video';
Expand Down Expand Up @@ -87,10 +87,10 @@ export const registerCoreBlocks = () => {
pullquote,
separator,
reusableBlock,
reusableTemplate,
spacer,
subhead,
table,
template,
textColumns,
verse,
video,
Expand Down
4 changes: 2 additions & 2 deletions packages/block-library/src/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,12 +33,12 @@ import * as nextpage from './nextpage';
import * as preformatted from './preformatted';
import * as pullquote from './pullquote';
import * as reusableBlock from './block';
import * as reusableTemplate from './template';
import * as separator from './separator';
import * as shortcode from './shortcode';
import * as spacer from './spacer';
import * as subhead from './subhead';
import * as table from './table';
import * as template from './template';
import * as textColumns from './text-columns';
import * as verse from './verse';
import * as video from './video';
Expand Down Expand Up @@ -76,10 +76,10 @@ export const registerCoreBlocks = () => {
pullquote,
separator,
reusableBlock,
reusableTemplate,
spacer,
subhead,
table,
template,
textColumns,
verse,
video,
Expand Down
1 change: 1 addition & 0 deletions test/integration/full-content/full-content.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -223,6 +223,7 @@ describe( 'full post content fixture', () => {
.map( ( block ) => block.name )
// We don't want tests for each oembed provider, which all have the same
// `save` functions and attributes.
// The `core/template` is not worth testing here because it's never saved, it's covered better in e2e tests.
.filter( ( name ) => name.indexOf( 'core-embed' ) !== 0 && name !== 'core/template' )
Copy link
Member

@mtias mtias Sep 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we update the comment here too? Not clear why we need to exclude it.

Copy link
Contributor Author

@youknowriad youknowriad Sep 12, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't want to add fixtures for this block. I thought this test would bring too much compared to the e2e test

.forEach( ( name ) => {
const nameToFilename = name.replace( /\//g, '__' );
Expand Down