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 View: Allow Escape key to deselect blocks if blocks are selected #48708

Merged
merged 11 commits into from
Sep 5, 2023
Merged

List View: Allow Escape key to deselect blocks if blocks are selected #48708

Changes from 9 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
7a77ce2
List View: Allow Escape key to deselect blocks if blocks are selected
andrewserong Mar 3, 2023
eb25cf2
Add comment, check which tab is active in post editor
andrewserong Mar 3, 2023
37625e8
Add speak call, use clearSelectedBlock action
andrewserong Mar 5, 2023
2589db5
Add behaviour to widgets editor, too
andrewserong Mar 8, 2023
afa979a
Move clear selection handling down to the List View container so that…
andrewserong Mar 9, 2023
01bb07b
Try selecting the last selected block when closing the list view if n…
andrewserong Mar 14, 2023
2c18117
Revert changes
andrewserong Jun 21, 2023
b93392d
Try clearing the selection from within the list view
andrewserong Jun 21, 2023
4d3cb50
Revert change
andrewserong Sep 1, 2023
068d08c
Add test
andrewserong Sep 1, 2023
0397d33
Fix issue where announcement for deselection was not occurring
andrewserong Sep 4, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
16 changes: 16 additions & 0 deletions packages/block-editor/src/components/list-view/block.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ import {
import { useDispatch, useSelect } from '@wordpress/data';
import { sprintf, __ } from '@wordpress/i18n';
import { focus } from '@wordpress/dom';
import { ESCAPE } from '@wordpress/keycodes';

/**
* Internal dependencies
Expand Down Expand Up @@ -148,6 +149,20 @@ function ListViewBlock( {
}
}, [] );

// If multiple blocks are selected, deselect all blocks when the user
// presses the escape key.
const onKeyDown = ( event ) => {
if (
event.keyCode === ESCAPE &&
! event.defaultPrevented &&
selectedClientIds.length > 0
) {
event.stopPropagation();
event.preventDefault();
selectBlock( event, undefined );
}
};

const onMouseEnter = useCallback( () => {
setIsHovered( true );
toggleBlockHighlight( clientId, true );
Expand Down Expand Up @@ -255,6 +270,7 @@ function ListViewBlock( {
return (
<ListViewLeaf
className={ classes }
onKeyDown={ onKeyDown }
onMouseEnter={ onMouseEnter }
onMouseLeave={ onMouseLeave }
onFocus={ onMouseEnter }
Expand Down