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.

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

Move HeadinglevelDropdown to its own component #46003

Merged
merged 36 commits into from
Jun 20, 2023
Merged
Show file tree
Hide file tree
Changes from 33 commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
a5fb915
Add a tag selection component for the block toolbar (replaces Heading…
carolinan Nov 23, 2022
753e2a8
Update index.php
carolinan Nov 24, 2022
939bd29
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Dec 16, 2022
165ad2f
update fixtures
carolinan Dec 16, 2022
d906d56
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Dec 30, 2022
787d080
Update CSS class, replace sprintf in the label with the tag name
carolinan Dec 30, 2022
6c62d43
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Jan 30, 2023
5691977
Add visible tag name to the drop down, add placeholder icon
carolinan Jan 30, 2023
1ba3734
Update README.md
carolinan Jan 30, 2023
a2d48fa
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Feb 8, 2023
5639693
Try to fix issue with the deprecation
carolinan Feb 8, 2023
48a7a10
Update CHANGELOG.md
carolinan Feb 8, 2023
98dbcde
Simplify and rename component
carolinan Feb 9, 2023
9a4bad3
Try reverting fixture changes
carolinan Feb 9, 2023
3724097
And again...
carolinan Feb 9, 2023
d331504
Update Query title to use the new component
carolinan Feb 9, 2023
1fba40d
Update Comments title
carolinan Feb 9, 2023
d397580
Update Site title with paragraph exception
carolinan Feb 9, 2023
dcf2f38
Update Heading block
carolinan Feb 9, 2023
62b1de0
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Feb 9, 2023
c2d4377
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Feb 10, 2023
33f32da
Update rich-text.test.js
carolinan Feb 10, 2023
a146a80
Block Heading Dropdown - Add support for mobile (#47967)
Feb 11, 2023
13476cf
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Feb 21, 2023
ed7000d
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Apr 12, 2023
b71e4df
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Apr 19, 2023
46b117a
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan May 12, 2023
a50c982
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan May 21, 2023
ec4278f
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan May 22, 2023
2716599
Text changes for a native file and e2e test spec
carolinan May 22, 2023
563f933
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan May 24, 2023
95866c5
Use the updated headingLevel icons
carolinan May 24, 2023
b674dc1
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Jun 14, 2023
b78b136
Update prop names
carolinan Jun 15, 2023
de0e2c9
Add 'options' property to type definiton
Mamaduka Jun 15, 2023
ce5c979
Merge branch 'trunk' into add/tag-selection-dropdown
carolinan Jun 20, 2023
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
5 changes: 5 additions & 0 deletions packages/block-editor/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

### Enhancements

- Add `HeadingLevelDropdown` component for selecting H1-H6 and paragraph HTML tags from the block toolbar.

## 12.3.0 (2023-06-07)

## 12.2.0 (2023-05-24)
Expand All @@ -18,6 +22,7 @@

## 11.8.0 (2023-04-12)


## 11.7.0 (2023-03-29)

- `ImageSizeControl`: Update image size label ([#49112](https://github.com/WordPress/gutenberg/pull/49112)).
Expand Down
12 changes: 12 additions & 0 deletions packages/block-editor/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -545,6 +545,18 @@ _Returns_

- `Object`: Typography block support derived CSS classes & styles.

### HeadingLevelDropdown

Dropdown for selecting a heading level (1 through 6) or paragraph (0).

_Parameters_

- _props_ `WPHeadingLevelDropdownProps`: Component props.

_Returns_

- `WPComponent`: The toolbar.

### HeightControl

HeightControl renders a linked unit control and range control for adjusting the height of a block.
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,52 @@
# Heading Level Dropdown

`<HeadingLevelDropdown>` Adds a dropdown for selecting H1-H6 and paragraph HTML tags from the block toolbar.
Uses `<ToolbarDropdownMenu>`.
Mamaduka marked this conversation as resolved.
Show resolved Hide resolved

## Usage

```jsx

import { BlockControls, HeadingLevelDropdown } from '@wordpress/block-editor';

const HEADING_LEVELS = [ 0, 1, 2, 3, 4, 5, 6 ];

const MyHeadingLevelToolbar = () => (
<BlockControls group="block">
<HeadingLevelDropdown
tags={ HEADING_LEVELS }
selectedTag={ tag }
onChange={ ( newTag ) =>
setAttributes( { tag: newTag } )
}
/>
</BlockControls>
);
```

### Props

#### tags

The list of available HTML tags, passed from the block.

- Type: `Object`
- Required: no

#### selectedTag

The chosen HTML tag.

- Type: `string`
- Required: no

#### onChange

Callback to run when toolbar value is changed.

- Type: `string`
- Required: yes

## Related components

Block Editor components are components that can be used to compose the UI of your block editor. Thus, they can only be used under a [`BlockEditorProvider`](https://github.com/WordPress/gutenberg/blob/HEAD/packages/block-editor/src/components/provider/README.md) in the components tree.
Original file line number Diff line number Diff line change
@@ -0,0 +1,45 @@
/**
* WordPress dependencies
*/
import {
headingLevel1,
headingLevel2,
headingLevel3,
headingLevel4,
headingLevel5,
headingLevel6,
paragraph,
} from '@wordpress/icons';

/** @typedef {import('@wordpress/element').WPComponent} WPComponent */

/**
* HeadingLevelIcon props.
*
* @typedef WPHeadingLevelIconProps
*
* @property {number} level The heading level to show an icon for.
*/

/**
* Heading level icon.
*
* @param {WPHeadingLevelIconProps} props Component props.
*
* @return {?WPComponent} The icon.
*/
export default function HeadingLevelIcon( { level } ) {
if ( level === 0 ) return paragraph;
const levelToPath = {
1: headingLevel1,
2: headingLevel2,
3: headingLevel3,
4: headingLevel4,
5: headingLevel5,
6: headingLevel6,
};
Mamaduka marked this conversation as resolved.
Show resolved Hide resolved
if ( ! levelToPath.hasOwnProperty( level ) ) {
return null;
}
return levelToPath[ level ];
}
Original file line number Diff line number Diff line change
Expand Up @@ -28,19 +28,26 @@ const POPOVER_PROPS = {
*/

/**
* Dropdown for selecting a heading level (1 through 6).
* Dropdown for selecting a heading level (1 through 6) or paragraph (0).
*
* @param {WPHeadingLevelDropdownProps} props Component props.
*
* @return {WPComponent} The toolbar.
*/
export default function HeadingLevelDropdown( { selectedLevel, onChange } ) {
export default function HeadingLevelDropdown( {
levels,
selectedLevel,
Mamaduka marked this conversation as resolved.
Show resolved Hide resolved
onChange,
} ) {
if ( levels === undefined ) {
levels = HEADING_LEVELS;
}
return (
<ToolbarDropdownMenu
popoverProps={ POPOVER_PROPS }
icon={ <HeadingLevelIcon level={ selectedLevel } /> }
label={ __( 'Change heading level' ) }
controls={ HEADING_LEVELS.map( ( targetLevel ) => {
label={ __( 'Change level' ) }
controls={ levels.map( ( targetLevel ) => {
{
const isActive = targetLevel === selectedLevel;

Expand All @@ -51,11 +58,14 @@ export default function HeadingLevelDropdown( { selectedLevel, onChange } ) {
isPressed={ isActive }
/>
),
label: sprintf(
// translators: %s: heading level e.g: "1", "2", "3"
__( 'Heading %d' ),
targetLevel
),
label:
targetLevel === 0
? __( 'Paragraph' )
: sprintf(
// translators: %s: heading level e.g: "1", "2", "3"
__( 'Heading %d' ),
targetLevel
),
isActive,
onClick() {
onChange( targetLevel );
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ export default function HeadingLevelDropdown( { selectedLevel, onChange } ) {
controls={ HEADING_LEVELS.map( ( index ) =>
createLevelControl( index, selectedLevel, onChange )
) }
label={ __( 'Change heading level' ) }
label={ __( 'Change level' ) }
/>
);
}
1 change: 1 addition & 0 deletions packages/block-editor/src/components/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ export { default as BlockEdit, useBlockEditContext } from './block-edit';
export { default as BlockIcon } from './block-icon';
export { default as BlockNavigationDropdown } from './block-navigation/dropdown';
export { default as BlockStyles } from './block-styles';
export { default as HeadingLevelDropdown } from './block-heading-level-dropdown';
export { default as __experimentalBlockVariationPicker } from './block-variation-picker';
export { default as __experimentalBlockPatternSetup } from './block-pattern-setup';
export { default as __experimentalBlockVariationTransforms } from './block-variation-transforms';
Expand Down
1 change: 1 addition & 0 deletions packages/block-editor/src/components/index.native.js
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ export {
JustifyContentControl,
} from './justify-content-control';
export { default as LineHeightControl } from './line-height-control';
export { default as HeadingLevelDropdown } from './block-heading-level-dropdown';
export { default as PlainText } from './plain-text';
export {
default as RichText,
Expand Down
6 changes: 1 addition & 5 deletions packages/block-library/src/comments-title/edit.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import {
useBlockProps,
InspectorControls,
store as blockEditorStore,
HeadingLevelDropdown,
} from '@wordpress/block-editor';
import { __, _n, sprintf } from '@wordpress/i18n';
import { useEntityProp } from '@wordpress/core-data';
Expand All @@ -21,11 +22,6 @@ import { useSelect } from '@wordpress/data';
import apiFetch from '@wordpress/api-fetch';
import { addQueryArgs } from '@wordpress/url';

/**
* Internal dependencies
*/
import HeadingLevelDropdown from '../heading/heading-level-dropdown';

export default function Edit( {
attributes: { textAlign, showPostTitle, showCommentsCount, level },
setAttributes,
Expand Down
2 changes: 1 addition & 1 deletion packages/block-library/src/heading/edit.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,12 +16,12 @@ import {
RichText,
useBlockProps,
store as blockEditorStore,
HeadingLevelDropdown,
} from '@wordpress/block-editor';

/**
* Internal dependencies
*/
import HeadingLevelDropdown from './heading-level-dropdown';
import { generateAnchor, setAnchor } from './autogenerate-anchors';

function HeadingEdit( {
Expand Down
4 changes: 2 additions & 2 deletions packages/block-library/src/post-title/edit.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@ import {
InspectorControls,
useBlockProps,
PlainText,
HeadingLevelDropdown,
privateApis as blockEditorPrivateApis,
} from '@wordpress/block-editor';
import { ToggleControl, TextControl, PanelBody } from '@wordpress/components';
Expand All @@ -22,7 +23,6 @@ import { useEntityProp } from '@wordpress/core-data';
/**
* Internal dependencies
*/
import HeadingLevelDropdown from '../heading/heading-level-dropdown';
import { useCanEditEntity } from '../utils/hooks';
import { unlock } from '../lock-unlock';

Expand All @@ -34,7 +34,7 @@ export default function PostTitleEdit( {
context: { postType, postId, queryId },
insertBlocksAfter,
} ) {
const TagName = 0 === level ? 'p' : 'h' + level;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I noticed that this code seems to have been adapted for allowing the post title to be a paragraph, but I don't remember this being implemented? It was only implemented for the site title.

Because of the discussions in this issue #30549
I have removed the partial support for p tag.

const TagName = 'h' + level;
const isDescendentOfQueryLoop = Number.isFinite( queryId );
/**
* Hack: useCanEditEntity may trigger an OPTIONS request to the REST API via the canUser resolver.
Expand Down
2 changes: 1 addition & 1 deletion packages/block-library/src/post-title/index.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@ function render_block_core_post_title( $attributes, $content, $block ) {

$tag_name = 'h2';
if ( isset( $attributes['level'] ) ) {
$tag_name = 0 === $attributes['level'] ? 'p' : 'h' . $attributes['level'];
$tag_name = 'h' . $attributes['level'];
}

if ( isset( $attributes['isLink'] ) && $attributes['isLink'] ) {
Expand Down
6 changes: 1 addition & 5 deletions packages/block-library/src/query-title/edit.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,15 +12,11 @@ import {
InspectorControls,
useBlockProps,
Warning,
HeadingLevelDropdown,
} from '@wordpress/block-editor';
import { ToggleControl, PanelBody } from '@wordpress/components';
import { __ } from '@wordpress/i18n';

/**
* Internal dependencies
*/
import HeadingLevelDropdown from '../heading/heading-level-dropdown';

const SUPPORTED_TYPES = [ 'archive', 'search' ];

export default function QueryTitleEdit( {
Expand Down
11 changes: 5 additions & 6 deletions packages/block-library/src/site-title/edit/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,13 @@ import {
InspectorControls,
BlockControls,
useBlockProps,
HeadingLevelDropdown,
} from '@wordpress/block-editor';
import { ToggleControl, PanelBody } from '@wordpress/components';
import { createBlock, getDefaultBlockName } from '@wordpress/blocks';
import { decodeEntities } from '@wordpress/html-entities';

/**
* Internal dependencies
*/
import LevelControl from './level-toolbar';
const HEADING_LEVELS = [ 0, 1, 2, 3, 4, 5, 6 ];

export default function SiteTitleEdit( {
attributes,
Expand Down Expand Up @@ -95,8 +93,9 @@ export default function SiteTitleEdit( {
return (
<>
<BlockControls group="block">
<LevelControl
level={ level }
<HeadingLevelDropdown
levels={ HEADING_LEVELS }
selectedLevel={ level }
onChange={ ( newLevel ) =>
setAttributes( { level: newLevel } )
}
Expand Down
28 changes: 0 additions & 28 deletions packages/block-library/src/site-title/edit/level-icon.js

This file was deleted.

34 changes: 0 additions & 34 deletions packages/block-library/src/site-title/edit/level-toolbar.js

This file was deleted.

Loading