Skip to content

1070 task list cta of the seo or readability child tasks land on the post seo readability analysis opened#23015

Open
vraja-pro wants to merge 17 commits intofeature/task-list-phase-3from
1070-task-list-cta-of-the-seo-or-readability-child-tasks-land-on-the-post-seo-readability-analysis-opened
Open

1070 task list cta of the seo or readability child tasks land on the post seo readability analysis opened#23015
vraja-pro wants to merge 17 commits intofeature/task-list-phase-3from
1070-task-list-cta-of-the-seo-or-readability-child-tasks-land-on-the-post-seo-readability-analysis-opened

Conversation

@vraja-pro
Copy link
Contributor

@vraja-pro vraja-pro commented Feb 23, 2026

Context

Summary

This PR can be summarized in the following changelog entry:

  • Opens the SEO analysis or the Readability analysis tabs in the post editor when clicking on the tasks to improve post Readability or SEO CTA buttons in the Task list.

Relevant technical choices:

Test instructions

Test instructions for the acceptance test before the PR gets merged

This PR can be acceptance tested by following these steps:

  • Have posts with focus keyphrase that are recent.
  • Use the block editor to edit your posts.
  • Go to Yoast SEO -> Task list
  • Click on the task to improve readability and click on one of the incomplete child tasks.
  • Check the CTA button label is Improve readability.
  • Check you are redirected to the post and the readability tab is open both in block editor sidebar and the metabox.
  • Check the focus is on the Readability tab.
  • Repeat for the SEO task and check the SEO analysis tab is open both on the block editor sidebar and the metabox.
  • Check the CTA button label is Improve SEO.
  • Check the focus is on the SEO analysis collapsible and it is open.
  • Enable Premium and repeat the check.
  • Disable block editor and use the classic editor and repeat the check.
  • Repeat the test for a post that is edited with elementor. The link would redirect to the post not the elementor editor, that expected, check the metabox has the analysis tab is open (either readability or seo).

Relevant test scenarios

  • Changes should be tested with the browser console open
  • Changes should be tested on different posts/pages/taxonomies/custom post types/custom taxonomies
  • Changes should be tested on different editors (Default Block/Gutenberg/Classic/Elementor/other)
  • Changes should be tested on different browsers
  • Changes should be tested on multisite

Test instructions for QA when the code is in the RC

  • QA should use the same steps as above.

QA can test this PR by following these steps:

Impact check

This PR affects the following parts of the plugin, which may require extra testing:

Other environments

  • This PR also affects Shopify. I have added a changelog entry starting with [shopify-seo], added test instructions for Shopify and attached the Shopify label to this PR.
  • This PR also affects Yoast SEO for Google Docs. I have added a changelog entry starting with [yoast-doc-extension], added test instructions for Yoast SEO for Google Docs and attached the Google Docs Add-on label to this PR.

Documentation

  • I have written documentation for this change. For example, comments in the Relevant technical choices, comments in the code, documentation on Confluence / shared Google Drive / Yoast developer portal, or other.

Quality assurance

  • I have tested this code to the best of my abilities.
  • During testing, I had activated all plugins that Yoast SEO provides integrations for.
  • I have added unit tests to verify the code works as intended.
  • If any part of the code is behind a feature flag, my test instructions also cover cases where the feature flag is switched off.
  • I have written this PR in accordance with my team's definition of done.
  • I have checked that the base branch is correctly set.
  • I have run grunt build:images and commited the results, if my PR introduces new images or SVGs.

Innovation

  • No innovation project is applicable for this PR.
  • This PR falls under an innovation project. I have attached the innovation label.
  • I have added my hours to the WBSO document.

Fixes https://github.com/Yoast/reserved-tasks/issues/1070

@vraja-pro vraja-pro added the changelog: non-user-facing Needs to be included in the 'Non-userfacing' category in the changelog label Feb 23, 2026
@vraja-pro vraja-pro added this to the feature/task-list-phase-3 milestone Feb 23, 2026
Copy link
Contributor

@FAMarfuaty FAMarfuaty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Because social appearance is set as initial open but we don't want the focus in that case.
@vraja-pro vraja-pro force-pushed the 1070-task-list-cta-of-the-seo-or-readability-child-tasks-land-on-the-post-seo-readability-analysis-opened branch from f65f2a0 to 605a4be Compare March 3, 2026 14:02
…he-seo-or-readability-child-tasks-land-on-the-post-seo-readability-analysis-opened
@coveralls
Copy link

coveralls commented Mar 3, 2026

Pull Request Test Coverage Report for Build cb7f498541079450d092741e23be6abfd122e9d4

Details

  • 16 of 37 (43.24%) changed or added relevant lines in 8 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.02%) to 53.893%

Changes Missing Coverage Covered Lines Changed/Added Lines %
packages/js/src/components/SidebarCollapsible.js 4 5 80.0%
packages/js/src/components/contentAnalysis/SeoAnalysis.js 0 1 0.0%
packages/js/src/components/contentAnalysis/ReadabilityAnalysis.js 0 2 0.0%
packages/js/src/initializers/block-editor-integration.js 0 3 0.0%
packages/js/src/initializers/metabox-tabs.js 0 14 0.0%
Files with Coverage Reduction New Missed Lines %
packages/js/src/components/contentAnalysis/SeoAnalysis.js 1 0.0%
packages/js/src/components/contentAnalysis/ReadabilityAnalysis.js 2 0.0%
Totals Coverage Status
Change from base Build 6cad32f98a761e1adcee035e55e0885f3ffc810e: -0.02%
Covered Lines: 34247
Relevant Lines: 63675

💛 - Coveralls
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates Task list child-task CTAs so that clicking “Improve readability” / “Improve SEO” deep-links into the post editor with the relevant Yoast analysis section opened and focused.

Changes:

  • Add a yoast-tab query parameter to the child-task CTA edit links for SEO/readability and update CTA labels.
  • In the block editor, auto-open the Yoast SEO sidebar when yoast-tab is present and expand the matching analysis panel.
  • In the classic editor metabox, handle yoast-tab by activating/focusing the matching metabox tab/collapsible.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
src/task-list/application/tasks/child-tasks/improve-content-seo-child.php Adds yoast-tab=seo to the CTA link and updates CTA label.
src/task-list/application/tasks/child-tasks/improve-content-readability-child.php Adds yoast-tab=readability to the CTA link and updates CTA label.
tests/Unit/Task_List/Application/Tasks/Improve_Content_SEO_Child/Improve_Content_SEO_Child_Call_To_Action_Test.php Updates unit test expectations for label and query-arg link building.
tests/Unit/Task_List/Application/Tasks/Improve_Content_Readability_Child/Improve_Content_Readability_Child_Call_To_Action_Test.php Updates unit test expectations for label and query-arg link building.
packages/js/src/initializers/metabox-tabs.js Adds URL param handling to focus/open the right metabox UI on load.
packages/js/src/initializers/block-editor-integration.js Opens Yoast sidebar in block editor when yoast-tab is present.
packages/js/src/components/contentAnalysis/SeoAnalysis.js Opens SEO analysis collapsible by default when yoast-tab=seo.
packages/js/src/components/contentAnalysis/ReadabilityAnalysis.js Opens Readability analysis collapsible by default when yoast-tab=readability (sidebar).
packages/js/src/components/SidebarCollapsible.js Adds initialIsOpen support and focuses the toggle when initially opened.
packages/js/src/components/MetaboxCollapsible.js Passes through initialIsOpen explicitly to the underlying component.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

vraja-pro and others added 3 commits March 4, 2026 15:54
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Aida Marfuaty <48715883+FAMarfuaty@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog: non-user-facing Needs to be included in the 'Non-userfacing' category in the changelog

4 participants