Skip to content

Commit e2a848e

Browse files
author
Jon Q
committed
Revert "Revert "Fullscreen Mode: Change the "Back" button to toggle the sidebar (#21121)" (#21929)"
This reverts commit caf52c1.
1 parent e110119 commit e2a848e

File tree

16 files changed

+231
-154
lines changed

16 files changed

+231
-154
lines changed

package-lock.json

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/e2e-tests/specs/editor/various/fullscreen-mode.test.js

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,5 @@ describe( 'Fullscreen Mode', () => {
2121
} );
2222

2323
expect( isFullscreenEnabled ).toBe( true );
24-
25-
const fullscreenCloseButton = await page.$(
26-
'.edit-post-fullscreen-mode-close'
27-
);
28-
29-
expect( fullscreenCloseButton ).not.toBeNull();
3024
} );
3125
} );

packages/edit-post/src/components/header/fullscreen-mode-close/index.js

Lines changed: 0 additions & 44 deletions
This file was deleted.

packages/edit-post/src/components/header/fullscreen-mode-close/style.scss

Lines changed: 0 additions & 29 deletions
This file was deleted.

packages/edit-post/src/components/header/index.js

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,11 @@ import { Button } from '@wordpress/components';
66
import { PostSavedState, PostPreviewButton } from '@wordpress/editor';
77
import { useSelect, useDispatch } from '@wordpress/data';
88
import { cog } from '@wordpress/icons';
9-
import { PinnedItems } from '@wordpress/interface';
9+
import { PinnedItems, AdminMenuToggle } from '@wordpress/interface';
1010

1111
/**
1212
* Internal dependencies
1313
*/
14-
import FullscreenModeClose from './fullscreen-mode-close';
1514
import HeaderToolbar from './header-toolbar';
1615
import MoreMenu from './more-menu';
1716
import PostPublishButtonOrToggle from './post-publish-button-or-toggle';
@@ -25,6 +24,7 @@ function Header( { onToggleInserter, isInserterOpen } ) {
2524
isPublishSidebarOpened,
2625
isSaving,
2726
getBlockSelectionStart,
27+
isFullscreenActive,
2828
} = useSelect(
2929
( select ) => ( {
3030
shortcut: select(
@@ -41,6 +41,9 @@ function Header( { onToggleInserter, isInserterOpen } ) {
4141
getBlockSelectionStart: select( 'core/block-editor' )
4242
.getBlockSelectionStart,
4343
isPostSaveable: select( 'core/editor' ).isEditedPostSaveable(),
44+
isFullscreenActive: select( 'core/edit-post' ).isFeatureActive(
45+
'fullscreenMode'
46+
),
4447
deviceType: select(
4548
'core/edit-post'
4649
).__experimentalGetPreviewDeviceType(),
@@ -62,7 +65,7 @@ function Header( { onToggleInserter, isInserterOpen } ) {
6265

6366
return (
6467
<div className="edit-post-header">
65-
<FullscreenModeClose />
68+
{ isFullscreenActive && <AdminMenuToggle /> }
6669
<div className="edit-post-header__toolbar">
6770
<HeaderToolbar
6871
onToggleInserter={ onToggleInserter }

packages/edit-post/src/style.scss

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ $footer-height: $button-size-small;
33
@import "../../interface/src/style.scss";
44

55
@import "./components/header/style.scss";
6-
@import "./components/header/fullscreen-mode-close/style.scss";
76
@import "./components/header/header-toolbar/style.scss";
87
@import "./components/header/more-menu/style.scss";
98
@import "./components/keyboard-shortcut-help-modal/style.scss";

packages/edit-site/src/components/header/fullscreen-mode-close/index.js

Lines changed: 0 additions & 35 deletions
This file was deleted.

packages/edit-site/src/components/header/fullscreen-mode-close/style.scss

Lines changed: 0 additions & 29 deletions
This file was deleted.

packages/edit-site/src/components/header/index.js

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,13 +9,12 @@ import {
99
__experimentalPreviewOptions as PreviewOptions,
1010
} from '@wordpress/block-editor';
1111
import { useSelect, useDispatch } from '@wordpress/data';
12-
import { PinnedItems } from '@wordpress/interface';
12+
import { PinnedItems, AdminMenuToggle } from '@wordpress/interface';
1313

1414
/**
1515
* Internal dependencies
1616
*/
1717
import { useEditorContext } from '../editor';
18-
import FullscreenModeClose from './fullscreen-mode-close';
1918
import MoreMenu from './more-menu';
2019
import TemplateSwitcher from '../template-switcher';
2120
import SaveButton from '../save-button';
@@ -52,6 +51,15 @@ export default function Header() {
5251
[]
5352
);
5453

54+
const { isFullscreenActive } = useSelect(
55+
( select ) => ( {
56+
isFullscreenActive: select( 'core/edit-site' ).isFeatureActive(
57+
'fullscreenMode'
58+
),
59+
} ),
60+
[]
61+
);
62+
5563
const deviceType = useSelect( ( select ) => {
5664
return select( 'core/edit-site' ).__experimentalGetPreviewDeviceType();
5765
}, [] );
@@ -62,7 +70,7 @@ export default function Header() {
6270

6371
return (
6472
<div className="edit-site-header">
65-
<FullscreenModeClose />
73+
{ isFullscreenActive && <AdminMenuToggle /> }
6674
<div className="edit-site-header__toolbar">
6775
<Inserter
6876
position="bottom right"

packages/edit-site/src/style.scss

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22

33
@import "./components/block-editor/style.scss";
44
@import "./components/header/style.scss";
5-
@import "./components/header/fullscreen-mode-close/style.scss";
65
@import "./components/header/more-menu/style.scss";
76
@import "./components/notices/style.scss";
87
@import "./components/sidebar/style.scss";

0 commit comments

Comments
 (0)