Skip to content

fix: styles for redirects page#22455

Merged
thijsoo merged 3 commits intofeature/redirection20from
bugfix/redirects20
Jul 25, 2025
Merged

fix: styles for redirects page#22455
thijsoo merged 3 commits intofeature/redirection20from
bugfix/redirects20

Conversation

@kyrylo-polozenko-newfold
Copy link
Copy Markdown
Contributor

@kyrylo-polozenko-newfold kyrylo-polozenko-newfold commented Jul 25, 2025

Context

Fixed styles for redirect page

Relevant technical choices:

UI changes

  • This PR changes the UI in the plugin. I have added the 'UI change' label to this PR.

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.

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.

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 #

@kyrylo-polozenko-newfold kyrylo-polozenko-newfold added UI change PRs that result in a change in the UI changelog: bugfix Needs to be included in the 'Bugfixes' category in the changelog labels Jul 25, 2025
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 25, 2025 •
edited
Loading

Pull Request Test Coverage Report for Build d328b4c32538cdd99749f223e2436c5490607597

Details

  • 3 of 10 (30.0%) changed or added relevant lines in 1 file are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage decreased (-0.5%) to 53.375%
Changes Missing Coverage Covered Lines Changed/Added Lines %
src/integrations/admin/redirects-page-integration.php 3 10 30.0%
Files with Coverage Reduction New Missed Lines %
src/integrations/admin/redirects-page-integration.php 1 45.45%
Totals Coverage Status
Change from base Build e79f37f179c8fd75266aff61a0dc2d7217f75b9f: -0.5%
Covered Lines: 29985
Relevant Lines: 57038
💛 - Coveralls

@thijsoo thijsoo added changelog: non-user-facing Needs to be included in the 'Non-userfacing' category in the changelog and removed UI change PRs that result in a change in the UI changelog: bugfix Needs to be included in the 'Bugfixes' category in the changelog labels Jul 25, 2025
@thijsoo thijsoo added this to the feature/redirection20 milestone Jul 25, 2025
thijsoo
thijsoo approved these changes Jul 25, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@thijsoo thijsoo left a comment

Choose a reason for hiding this comment

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

CR + ACC 👍

@thijsoo thijsoo merged commit 5fdfee9 into feature/redirection20 Jul 25, 2025
26 checks passed
@thijsoo thijsoo deleted the bugfix/redirects20 branch July 25, 2025 10:34
@vraja-pro vraja-pro mentioned this pull request Jul 28, 2025
Merged
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@thijsoo thijsoo thijsoo approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

25.7

Development

Successfully merging this pull request may close these issues.

4 participants

@kyrylo-polozenko-newfold @coveralls @thijsoo @enricobattocchi
⚡