Skip to content

Commit f620c38

Browse files
Mamadukat-hamanoadamsilversteindesrosj
authored andcommitted
Notes: Remove the pin/unpin button from the sidebar header (#72825)
Co-authored-by: Mamaduka <[email protected]> Co-authored-by: t-hamano <[email protected]> Co-authored-by: adamsilverstein <[email protected]> Co-authored-by: desrosj <[email protected]>
1 parent ad996fb commit f620c38

File tree

2 files changed

+5
-24
lines changed

2 files changed

+5
-24
lines changed

packages/editor/src/components/collab-sidebar/index.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -157,6 +157,11 @@ function NotesSidebar( { postId, mode } ) {
157157
identifier={ collabHistorySidebarName }
158158
name={ collabHistorySidebarName }
159159
title={ __( 'Notes' ) }
160+
header={
161+
<h2 className="interface-complementary-area-header__title">
162+
{ __( 'Notes' ) }
163+
</h2>
164+
}
160165
icon={ commentIcon }
161166
closeLabel={ __( 'Close Notes' ) }
162167
>

test/e2e/specs/editor/various/block-comments.spec.js

Lines changed: 0 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -19,30 +19,6 @@ test.describe( 'Block Comments', () => {
1919
await requestUtils.deleteAllComments( 'note' );
2020
} );
2121

22-
test( 'can pin and unpin comments sidebar', async ( {
23-
editor,
24-
page,
25-
blockCommentUtils,
26-
} ) => {
27-
const topBarButton = await blockCommentUtils.openBlockCommentSidebar();
28-
await page
29-
.getByRole( 'button', { name: 'Unpin from toolbar' } )
30-
.click();
31-
await expect( topBarButton ).toBeHidden();
32-
33-
await editor.openDocumentSettingsSidebar();
34-
await page
35-
.getByRole( 'region', {
36-
name: 'Editor top bar',
37-
} )
38-
.getByRole( 'button', { name: 'Options' } )
39-
.click();
40-
await page.getByRole( 'menuitemcheckbox', { name: 'Notes' } ).click();
41-
await page.getByRole( 'button', { name: 'Pin to toolbar' } ).click();
42-
43-
await expect( topBarButton ).toBeVisible();
44-
} );
45-
4622
test( 'should move focus to add a new note form', async ( {
4723
editor,
4824
page,

0 commit comments

Comments
 (0)