@elementor/editor-site-navigation 0.10.2 → 0.11.0

package/CHANGELOG.md

@@ -3,6 +3,25 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [0.11.0](https://github.com/elementor/elementor-packages/compare/@elementor/editor-site-navigation@0.10.3...@elementor/editor-site-navigation@0.11.0) (2023-07-24)
+
+
+### Features
+
+* **pages-panel:** change env key name in pages panel [ED-11514] ([#92](https://github.com/elementor/elementor-packages/issues/92)) ([52bf226](https://github.com/elementor/elementor-packages/commit/52bf22632bea544b69c0d58c4383c33b8d1c4aa8))
+
+
+
+
+
+## [0.10.3](https://github.com/elementor/elementor-packages/compare/@elementor/editor-site-navigation@0.10.2...@elementor/editor-site-navigation@0.10.3) (2023-07-17)
+
+**Note:** Version bump only for package @elementor/editor-site-navigation
+
+
+
+
+
 ## [0.10.2](https://github.com/elementor/elementor-packages/compare/@elementor/editor-site-navigation@0.10.1...@elementor/editor-site-navigation@0.10.2) (2023-07-17)
 
 **Note:** Version bump only for package @elementor/editor-site-navigation

package/dist/index.js

@@ -645,7 +645,7 @@ var { env, validateEnv } = (0, import_env.parseEnv)("@elementor/editor-site-navi
 // src/init.ts
 function init() {
   registerTopBarMenuItems();
-  if (env.is_panel_active) {
+  if (env.is_pages_panel_active) {
     (0, import_editor_panels3.registerPanel)(panel);
     registerButton();
   }
@@ -653,7 +653,7 @@ function init() {
 function registerTopBarMenuItems() {
   (0, import_editor_app_bar.injectIntoPageIndication)({
     id: "document-recently-edited",
-    filler: RecentlyEdited
+    component: RecentlyEdited
   });
 }
 function registerButton() {

package/dist/index.mjs

@@ -636,7 +636,7 @@ var { env, validateEnv } = parseEnv("@elementor/editor-site-navigation", (envDat
 // src/init.ts
 function init() {
   registerTopBarMenuItems();
-  if (env.is_panel_active) {
+  if (env.is_pages_panel_active) {
     registerPanel(panel);
     registerButton();
   }
@@ -644,7 +644,7 @@ function init() {
 function registerTopBarMenuItems() {
   injectIntoPageIndication({
     id: "document-recently-edited",
-    filler: RecentlyEdited
+    component: RecentlyEdited
   });
 }
 function registerButton() {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@elementor/editor-site-navigation",
-	"version": "0.10.2",
+	"version": "0.11.0",
 	"private": false,
 	"author": "Elementor Team",
 	"homepage": "https://elementor.com/",
@@ -32,9 +32,9 @@
 		"dev": "tsup --config=../../tsup.dev.ts"
 	},
 	"dependencies": {
-		"@elementor/editor-app-bar": "^0.6.8",
+		"@elementor/editor-app-bar": "^0.6.9",
 		"@elementor/editor-documents": "^0.8.3",
-		"@elementor/editor-panels": "^0.1.4",
+		"@elementor/editor-panels": "^0.1.5",
 		"@elementor/env": "^0.2.0",
 		"@elementor/icons": "^0.6.1",
 		"@elementor/query": "^0.1.1",
@@ -49,5 +49,5 @@
 	"elementor": {
 		"type": "extension"
 	},
-	"gitHead": "fbb55b97a2581af49559479b6abf63f5acbfc28b"
+	"gitHead": "2cd16cef577fdeafc697fc3ce1a3cfc7e17b569b"
 }

package/src/components/panel/pages-list/__tests__/page-list-item.test.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { act, render } from '@testing-library/react';
+import { act, render, screen } from '@testing-library/react';
 import { Post } from '../../../../types';
 import PageListItem from '../page-list-item';
 import { useNavigateToDocument } from '@elementor/editor-documents';
@@ -27,14 +27,14 @@ describe( '@elementor/editor-site-navigation - PageListItem', () => {
 		};
 
 		// Act.
-		const { getByText, queryByText } = render( <PageListItem page={ page } /> );
+		render( <PageListItem page={ page } /> );
 
 		// Assert.
-		const label = getByText( 'Test Page' );
-		const publishedLabel = queryByText( 'publish', { exact: false } );
+		const label = screen.getByText( 'Test Page' );
+		const publishedLabel = screen.queryByText( 'publish', { exact: false } );
 
 		expect( label ).toBeInTheDocument();
-		expect( publishedLabel ).toBeNull();
+		expect( publishedLabel ).not.toBeInTheDocument();
 	} );
 
 	it( 'should show the page status for non-published pages', () => {
@@ -50,10 +50,10 @@ describe( '@elementor/editor-site-navigation - PageListItem', () => {
 		};
 
 		// Act.
-		const { getByText } = render( <PageListItem page={ page } /> );
+		render( <PageListItem page={ page } /> );
 
 		// Assert.
-		const label = getByText( 'draft', { exact: false } );
+		const label = screen.getByText( 'draft', { exact: false } );
 		expect( label ).toBeInTheDocument();
 	} );
 
@@ -72,8 +72,8 @@ describe( '@elementor/editor-site-navigation - PageListItem', () => {
 		const actions = [ 'View Page', 'Rename', 'Duplicate', 'Delete', 'Set as homepage' ];
 
 		// Act.
-		const { getByText, getAllByRole } = render( <PageListItem page={ page } /> );
-		const buttons = getAllByRole( 'button' );
+		render( <PageListItem page={ page } /> );
+		const buttons = screen.getAllByRole( 'button' );
 		// Button to open menu
 		const button = buttons[ 1 ];
 
@@ -84,7 +84,7 @@ describe( '@elementor/editor-site-navigation - PageListItem', () => {
 
 		// Assert.
 		actions.forEach( ( action ) => {
-			const label = getByText( action );
+			const label = screen.getByText( action );
 			expect( label ).toBeInTheDocument();
 		} );
 	} );
@@ -107,8 +107,8 @@ describe( '@elementor/editor-site-navigation - PageListItem', () => {
 		};
 
 		// Act.
-		const { getAllByRole } = render( <PageListItem page={ page } /> );
-		const buttons = getAllByRole( 'button' );
+		render( <PageListItem page={ page } /> );
+		const buttons = screen.getAllByRole( 'button' );
 		const button = buttons[ 0 ];
 
 		act( () => {

package/src/components/panel/pages-list/__tests__/pages-collapsible-list.test.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { render } from '@testing-library/react';
+import { render, screen } from '@testing-library/react';
 import PagesCollapsibleList from '../pages-collapsible-list';
 
 jest.mock( '../../../../hooks/use-posts', () => ( {
@@ -29,29 +29,29 @@ describe( '@elementor/editor-site-navigation - PagesCollapsibleList', () => {
 
 	it( 'should render closed list', () => {
 		// Act.
-		const { getByText, queryByText } = render(
+		render(
 			<PagesCollapsibleList isOpenByDefault={ false } />,
 		);
 
 		// Assert.
-		const label = getByText( `Pages (3)` );
+		const label = screen.getByText( `Pages (3)` );
 		expect( label ).toBeInTheDocument();
 
-		const postInList = queryByText( 'Services' );
-		expect( postInList ).toBeNull();
+		const postInList = screen.queryByText( 'Services' );
+		expect( postInList ).not.toBeInTheDocument();
 	} );
 
 	it( 'should render open list', () => {
 		// Act.
-		const { getByText } = render(
+		render(
 			<PagesCollapsibleList isOpenByDefault={ true } />,
 		);
 
 		// Assert.
-		const label = getByText( `Pages (3)` );
+		const label = screen.getByText( `Pages (3)` );
 		expect( label ).toBeInTheDocument();
 
-		const postInList = getByText( 'Services' );
+		const postInList = screen.getByText( 'Services' );
 		expect( postInList ).toBeInTheDocument();
 	} );
 } );

package/src/components/top-bar/__tests__/add-new-page.test.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { fireEvent, render, waitFor } from '@testing-library/react';
+import { fireEvent, render, screen, waitFor } from '@testing-library/react';
 import { useHostDocument, useActiveDocument, useNavigateToDocument } from '@elementor/editor-documents';
 import RecentlyEdited from '../recently-edited';
 import { createMockDocument } from 'test-utils';
@@ -45,14 +45,14 @@ describe( '@elementor/recently-edited - Top bar add new page', () => {
 
 		jest.mocked( useRecentPosts ).mockReturnValue( { isLoading, recentPosts } );
 
-		const { getByText, getAllByRole } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Act.
-		const buttons = getAllByRole( 'button' );
+		const buttons = screen.getAllByRole( 'button' );
 		fireEvent.click( buttons[ 0 ] ); // Opens the recently edited menu
 
 		// Assert.
-		const label = getByText( 'Add new page', { exact: false } );
+		const label = screen.getByText( 'Add new page', { exact: false } );
 		expect( label ).toBeInTheDocument();
 	} );
 
@@ -69,22 +69,21 @@ describe( '@elementor/recently-edited - Top bar add new page', () => {
 		jest.mocked( useCreatePage ).mockReturnValue( { isLoading, create } );
 		jest.mocked( useNavigateToDocument ).mockReturnValue( navigateToDocument );
 
-		const { getByText, getAllByRole } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Act.
-		const buttons = getAllByRole( 'button' );
+		const buttons = screen.getAllByRole( 'button' );
 		fireEvent.click( buttons[ 0 ] ); // Opens the recently edited menu
 
-		const addNewPage = getByText( 'Add new page', { exact: false } );
+		const addNewPage = screen.getByText( 'Add new page', { exact: false } );
 		fireEvent.click( addNewPage );
 
 		// Assert.
-		await waitFor( () => {
-			expect( create ).toHaveBeenCalledTimes( 1 );
+		expect( create ).toHaveBeenCalledTimes( 1 );
 
-			expect( navigateToDocument ).toHaveBeenCalledTimes( 1 );
-			expect( navigateToDocument ).toHaveBeenCalledWith( 123 );
-		} );
+		await waitFor( () => expect( navigateToDocument ).toHaveBeenCalledTimes( 1 ) );
+
+		expect( navigateToDocument ).toHaveBeenCalledWith( 123 );
 	} );
 } );
 

package/src/components/top-bar/__tests__/recently-edited.test.tsx

@@ -1,5 +1,5 @@
 import * as React from 'react';
-import { fireEvent, render } from '@testing-library/react';
+import { fireEvent, render, screen } from '@testing-library/react';
 import { useHostDocument, useActiveDocument, useNavigateToDocument } from '@elementor/editor-documents';
 import RecentlyEdited from '../recently-edited';
 import { createMockDocument } from 'test-utils';
@@ -31,11 +31,11 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 
 	it( 'should show the title of the active document without its status when the document is published', async () => {
 		// Act.
-		const { queryByText } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Assert.
-		expect( queryByText( 'Active Document' ) ).toBeTruthy();
-		expect( queryByText( '(publish)' ) ).not.toBeTruthy();
+		expect( screen.getByText( 'Active Document' ) ).toBeInTheDocument();
+		expect( screen.queryByText( '(publish)' ) ).not.toBeInTheDocument();
 	} );
 
 	it( 'should show the title of the active document with its status when the document is not published', async () => {
@@ -52,11 +52,11 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 		);
 
 		// Act.
-		const { queryByText } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Assert.
-		expect( queryByText( 'Active Document' ) ).toBeTruthy();
-		expect( queryByText( '(Draft)' ) ).toBeTruthy();
+		expect( screen.getByText( 'Active Document' ) ).toBeInTheDocument();
+		expect( screen.getByText( '(Draft)' ) ).toBeInTheDocument();
 	} );
 
 	it( 'should show the title of the host document when there is no active document', () => {
@@ -64,10 +64,10 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 		jest.mocked( useActiveDocument ).mockImplementation( () => null );
 
 		// Act.
-		const { queryByText } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Assert.
-		expect( queryByText( 'Host Document' ) ).toBeTruthy();
+		expect( screen.getByText( 'Host Document' ) ).toBeInTheDocument();
 	} );
 
 	it( 'should show the title of the host document when the active document is kit', () => {
@@ -85,10 +85,10 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 		);
 
 		// Act.
-		const { queryByText } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Assert.
-		expect( queryByText( 'Host Document' ) ).toBeTruthy();
+		expect( screen.getByText( 'Host Document' ) ).toBeInTheDocument();
 	} );
 
 	it( 'should show nothing if there are no documents', () => {
@@ -97,11 +97,11 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 		jest.mocked( useHostDocument ).mockImplementation( () => null );
 
 		// Act.
-		const { queryByText } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Assert.
-		expect( queryByText( 'Host Document' ) ).not.toBeTruthy();
-		expect( queryByText( 'Active Document' ) ).not.toBeTruthy();
+		expect( screen.queryByText( 'Host Document' ) ).not.toBeInTheDocument();
+		expect( screen.queryByText( 'Active Document' ) ).not.toBeInTheDocument();
 	} );
 
 	it( 'should show empty state', () => {
@@ -122,14 +122,14 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 
 		jest.mocked( useRecentPosts ).mockReturnValue( { isLoading, recentPosts } );
 
-		const { getByText, getAllByRole } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Act.
-		const buttons = getAllByRole( 'button' );
+		const buttons = screen.getAllByRole( 'button' );
 		fireEvent.click( buttons[ 0 ] ); // Opens the recently edited menu
 
 		// Assert.
-		const label = getByText( 'There are no other pages or templates on this site yet', { exact: false } );
+		const label = screen.getByText( 'There are no other pages or templates on this site yet', { exact: false } );
 		expect( label ).toBeInTheDocument();
 	} );
 
@@ -161,20 +161,20 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 
 		jest.mocked( useRecentPosts ).mockReturnValue( { isLoading, recentPosts } );
 
-		const { getByText, getByRole, getAllByRole } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Act.
-		const buttons = getAllByRole( 'button' );
+		const buttons = screen.getAllByRole( 'button' );
 		fireEvent.click( buttons[ 0 ] ); // Opens the recently edited menu
 
 		// Assert.
-		const menu = getByRole( 'menu' );
+		const menu = screen.getByRole( 'menu' );
 		expect( menu ).toBeInTheDocument();
 
-		const label = getByText( 'Recent' );
+		const label = screen.getByText( 'Recent' );
 		expect( label ).toBeInTheDocument();
 
-		expect( getByText( 'Test post' ) ).toBeInTheDocument();
+		expect( screen.getByText( 'Test post' ) ).toBeInTheDocument();
 	} );
 
 	it( 'should render titles with HTML entities', () => {
@@ -219,17 +219,17 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 		jest.mocked( useRecentPosts ).mockReturnValue( { isLoading, recentPosts } );
 
 		// Act.
-		const { getByText, getByRole } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Assert - the document title should be rendered with the HTML entity.
-		expect( getByText( 'Header title with special char ¥' ) ).toBeInTheDocument();
+		expect( screen.getByText( 'Header title with special char ¥' ) ).toBeInTheDocument();
 
 		// Open the posts list.
-		fireEvent.click( getByRole( 'button' ) );
+		fireEvent.click( screen.getByRole( 'button' ) );
 
 		// Assert - the post title should be rendered with the HTML entity.
-		expect( getByText( 'Post title with <h1>HTML</h1>' ) ).toBeInTheDocument();
-		expect( getByText( 'Post title with <HTML entities>' ) ).toBeInTheDocument();
+		expect( screen.getByText( 'Post title with <h1>HTML</h1>' ) ).toBeInTheDocument();
+		expect( screen.getByText( 'Post title with <HTML entities>' ) ).toBeInTheDocument();
 	} );
 
 	it( 'should navigate to document on click', () => {
@@ -253,13 +253,13 @@ describe( '@elementor/recently-edited - Top bar Recently Edited', () => {
 			} ],
 		} );
 
-		const { getByText, getByRole } = render( <RecentlyEdited /> );
+		render( <RecentlyEdited /> );
 
 		// Open the posts list.
-		fireEvent.click( getByRole( 'button' ) );
+		fireEvent.click( screen.getByRole( 'button' ) );
 
 		// Act.
-		fireEvent.click( getByText( 'Test post' ) );
+		fireEvent.click( screen.getByText( 'Test post' ) );
 
 		// Assert.
 		expect( navigateToDocument ).toHaveBeenCalledTimes( 1 );

package/src/env.ts

@@ -1,9 +1,9 @@
 import { parseEnv } from '@elementor/env';
 
 export const { env, validateEnv } = parseEnv<{
-	is_panel_active: boolean;
+	is_pages_panel_active: boolean;
 }>( '@elementor/editor-site-navigation', ( envData ) => {
 	return envData as {
-		is_panel_active: boolean;
+		is_pages_panel_active: boolean;
 	};
 } );

package/src/hooks/__tests__/use-create-page.test.ts

@@ -1,4 +1,4 @@
-import { waitFor, renderHook } from '@testing-library/react';
+import { renderHook } from '@testing-library/react';
 import apiFetch from '@wordpress/api-fetch';
 import useCreatePage, { endpointPath } from '../use-create-page';
 
@@ -29,13 +29,11 @@ describe( '@elementor/recently-edited/use-page', () => {
 		create();
 
 		// Assert.
-		await waitFor( () => {
-			expect( apiFetch ).toHaveBeenCalledWith( {
-				data: { post_type: 'page' },
-				method: 'POST',
-				path: endpointPath,
-			} );
-			expect( apiFetch ).toHaveBeenCalledTimes( 1 );
+		expect( apiFetch ).toHaveBeenCalledTimes( 1 );
+		expect( apiFetch ).toHaveBeenCalledWith( {
+			data: { post_type: 'page' },
+			method: 'POST',
+			path: endpointPath,
 		} );
 	} );
 } );

package/src/hooks/__tests__/use-posts.test.ts

@@ -29,12 +29,10 @@ describe( '@elementor/site-settings/use-posts', () => {
 		// Assert.
 		const expectedPath = `/wp/v2/pages?status=any&per_page=-1&_fields=${ encodeURIComponent( 'id,type,title,link,status' ) }`;
 
-		await waitFor( () => {
-			expect( apiFetch ).toHaveBeenCalledWith( {
-				path: expectedPath,
-			} );
-			expect( apiFetch ).toHaveBeenCalledTimes( 1 );
+		expect( apiFetch ).toHaveBeenCalledWith( {
+			path: expectedPath,
 		} );
+		expect( apiFetch ).toHaveBeenCalledTimes( 1 );
 
 		await waitFor( () => {
 			return component.result.current.isSuccess;

package/src/hooks/__tests__/use-recent-posts.test.ts

@@ -30,11 +30,11 @@ describe( 'useRecentPosts', () => {
 
 		const { result } = renderHook( () => useRecentPosts( 1 ) );
 
-		await waitFor( () => {
-			expect( apiFetch ).toHaveBeenCalledWith( {
-				path: endpointPath + '?posts_per_page=5&post__not_in=1',
-			} );
+		expect( apiFetch ).toHaveBeenCalledWith( {
+			path: endpointPath + '?posts_per_page=5&post__not_in=1',
+		} );
 
+		await waitFor( () => {
 			expect( result.current.recentPosts ).toBe( posts );
 		} );
 	} );

package/src/init.ts

@@ -8,7 +8,7 @@ import { env } from './env';
 export default function init() {
 	registerTopBarMenuItems();
 	// TODO 06/06/2023 :  remove if when we are production ready
-	if ( env.is_panel_active ) {
+	if ( env.is_pages_panel_active ) {
 		registerPanel( panel );
 		registerButton();
 	}
@@ -17,7 +17,7 @@ export default function init() {
 function registerTopBarMenuItems() {
 	injectIntoPageIndication( {
 		id: 'document-recently-edited',
-		filler: RecentlyEdited,
+		component: RecentlyEdited,
 	} );
 }