Skip to content

Commit 2cec5ca

Browse files
Internal: Revert changes to quicklinks [TMZ-923] (#545) (#546)
1 parent f692686 commit 2cec5ca

File tree

2 files changed

+3
-9
lines changed

2 files changed

+3
-9
lines changed

modules/admin-home/rest/admin-config.php

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ private function get_elementor_editor_url( ?int $page_id, string $active_tab ):
8686
'post' => $page_id,
8787
'action' => 'elementor',
8888
'active-tab' => $active_tab,
89-
'active-document' => $active_kit_id,
9089
],
9190
admin_url( 'post.php' )
9291
);
@@ -285,18 +284,18 @@ public function get_quicklinks( $config, ?int $elementor_page_id = null ): array
285284
$config['quickLinks'] = [
286285
'site_name' => [
287286
'title' => __( 'Site Name', 'hello-elementor' ),
288-
'link' => $this->get_open_homepage_with_tab( $elementor_page_id, 'settings-site-identity' ),
287+
'link' => $this->get_open_homepage_with_tab( $elementor_page_id, 'settings-site-identity', null, [ 'autofocus[section]' => 'title_tagline' ] ),
289288
'icon' => 'TextIcon',
290289

291290
],
292291
'site_logo' => [
293292
'title' => __( 'Site Logo', 'hello-elementor' ),
294-
'link' => $this->get_open_homepage_with_tab( $elementor_page_id, 'settings-site-identity' ),
293+
'link' => $this->get_open_homepage_with_tab( $elementor_page_id, 'settings-site-identity', null, [ 'autofocus[section]' => 'title_tagline' ] ),
295294
'icon' => 'PhotoIcon',
296295
],
297296
'site_favicon' => [
298297
'title' => __( 'Site Favicon', 'hello-elementor' ),
299-
'link' => $this->get_open_homepage_with_tab( $elementor_page_id, 'settings-site-identity' ),
298+
'link' => $this->get_open_homepage_with_tab( $elementor_page_id, 'settings-site-identity', null, [ 'autofocus[section]' => 'title_tagline' ] ),
300299
'icon' => 'AppsIcon',
301300
],
302301
];

tests/playwright/tests/admin/hello-theme-admin-home.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -63,11 +63,6 @@ test.describe( 'Hello Theme Admin Home Page', () => {
6363
const linkElement = page.locator( `h6:has-text("${ linkTest.linkText }") a` );
6464
await expect( linkElement ).toBeVisible();
6565

66-
if ( !! process.env.DAILY_MATRIX_WORKFLOW ) {
67-
// Skip this test in Daily Matrix workflow due to artifact deployment timing issues
68-
continue;
69-
}
70-
7166
await Promise.all( [
7267
page.waitForURL( linkTest.expectedUrlPattern ),
7368
linkElement.click(),

0 commit comments

Comments
 (0)