Skip to content
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

Jump to bottom

List: use nested blocks #42711

Merged
merged 17 commits into from
Aug 24, 2022
Merged

List: use nested blocks #42711

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
24a232c
list: use nested blocks
ellatrix Aug 12, 2022
aa0dc92
Restore example
ellatrix Aug 12, 2022
b395901
Fix rebase
ellatrix Aug 13, 2022
87a0d51
Fix a few tests
ellatrix Aug 13, 2022
2901dcb
Fix list e2e tests
ellatrix Aug 13, 2022
a8bac56
Fix splitting merging e2e tests
ellatrix Aug 13, 2022
92f64bd
wip
ellatrix Aug 17, 2022
89619ef
Fix select all
ellatrix Aug 21, 2022
1b58454
Fix remaining list tests
ellatrix Aug 21, 2022
2efba02
Fix remaining tests
ellatrix Aug 21, 2022
a04be15
Remove obsolete snapshot
ellatrix Aug 21, 2022
e046d45
Fix rebase problem
ellatrix Aug 22, 2022
fea69e8
Restore type attribute
ellatrix Aug 22, 2022
8da58c0
Update integration tests for type attr
ellatrix Aug 23, 2022
f8963bc
Update lib/experimental/block-editor-settings-mobile.php
ellatrix Aug 23, 2022
64b8794
Restore anchor in raw transform
ellatrix Aug 23, 2022
30c6b5b
Work around e2e test issue
ellatrix Aug 24, 2022
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
Restore anchor in raw transform
  • Loading branch information
@ellatrix @oandregal
ellatrix authored and oandregal committed Aug 24, 2022
commit 64b8794a3c839faeb261153a16eff47f3073cf32
6 changes: 1 addition & 5 deletions packages/block-library/src/list/save.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,11 +7,7 @@ export default function save( { attributes } ) {
const { ordered, type, reversed, start } = attributes;
const TagName = ordered ? 'ol' : 'ul';
return (
<TagName
reversed={ reversed }
start={ start }
{ ...useBlockProps.save( { type, reversed, start } ) }
>
<TagName { ...useBlockProps.save( { type, reversed, start } ) }>
<InnerBlocks.Content />
</TagName>
);
Expand Down
1 change: 1 addition & 0 deletions packages/block-library/src/list/utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@ import { createBlock } from '@wordpress/blocks';
export function createListBlockFromDOMElement( listElement ) {
const listAttributes = {
ordered: 'OL' === listElement.tagName,
anchor: listElement.id === '' ? undefined : listElement.id,
start: listElement.getAttribute( 'start' )
? parseInt( listElement.getAttribute( 'start' ), 10 )
: undefined,
Expand Down
4 changes: 2 additions & 2 deletions test/integration/__snapshots__/blocks-raw-handling.test.js.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -127,9 +127,9 @@ exports[`rawHandler should convert a caption shortcode with link 1`] = `

exports[`rawHandler should convert a list with attributes 1`] = `
"<!-- wp:list {\\"ordered\\":true,\\"type\\":\\"i\\",\\"start\\":2,\\"reversed\\":true} -->
<ol reversed start=\\"2\\" type=\\"i\\"><!-- wp:list-item -->
<ol type=\\"i\\" reversed start=\\"2\\"><!-- wp:list-item -->
<li>1<!-- wp:list {\\"ordered\\":true,\\"type\\":\\"i\\",\\"start\\":2,\\"reversed\\":true} -->
<ol reversed start=\\"2\\" type=\\"i\\"><!-- wp:list-item -->
<ol type=\\"i\\" reversed start=\\"2\\"><!-- wp:list-item -->
<li>1</li>
<!-- /wp:list-item --></ol>
<!-- /wp:list --></li>
Expand Down