Skip to content

Comments

Componentry: remove Jetpack color overrides on core components#47317

Open
simison wants to merge 2 commits intotrunkfrom
update/jp-components-core-colors
Open

Componentry: remove Jetpack color overrides on core components#47317
simison wants to merge 2 commits intotrunkfrom
update/jp-components-core-colors

Conversation

@simison
Copy link
Member

@simison simison commented Feb 25, 2026

Spin-off from #47056

Jetpack components mostly rely on core components under the hood, with added features and style overrides on spacing/colors.

This PR focuses on removing color overrides everywhere where we used core components under the hood. I'll do functional changes and straight component-replacements in separate PR so that this can be tested easier.

Proposed changes:

Other information:

  • Have you written new tests for your changes, if applicable?
  • Have you checked the E2E test CI results, and verified that your changes do not break them?
  • Have you tested your changes on WordPress.com, if applicable (if so, you'll see a generated comment below with a script to run)?

Jetpack product discussion

Does this pull request change what data or activity we track or use?

Testing instructions:

  • Go through different pages in Jetpack main plugin and individual plugins for Social, Boost, VideoPress and Protect.
  • Search & Boots dashboards had the most significant CSS removals, so those need testing thoroughly.
  • Pay attention especially to toggle, button and input componentry. Note hover/focus colors.

Changelog

  • Generate changelog entries for this PR (using AI).

Copilot AI review requested due to automatic review settings February 25, 2026 12:42
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

Are you an Automattician? Please test your changes on all WordPress.com environments to help mitigate accidental explosions.

  • To test on WoA, go to the Plugins menu on a WoA dev site. Click on the "Upload" button and follow the upgrade flow to be able to upload, install, and activate the Jetpack Beta plugin. Once the plugin is active, go to Jetpack > Jetpack Beta, select your plugin (Jetpack or WordPress.com Site Helper), and enable the update/jp-components-core-colors branch.
  • To test on Simple, run the following command on your sandbox:
bin/jetpack-downloader test jetpack update/jp-components-core-colors
bin/jetpack-downloader test jetpack-mu-wpcom-plugin update/jp-components-core-colors

Interested in more tips and information?

  • In your local development environment, use the jetpack rsync command to sync your changes to a WoA dev blog.
  • Read more about our development workflow here: PCYsg-eg0-p2
  • Figure out when your changes will be shipped to customers here: PCYsg-eg5-p2

@github-actions github-actions bot added [Feature] Publicize Now Jetpack Social, auto-sharing [JS Package] Base Styles [Package] My Jetpack [Package] Publicize [Package] Search Contains core Search functionality for Jetpack and Search plugins [Package] VideoPress [Plugin] Boost A feature to speed up the site and improve performance. [Plugin] Protect A plugin with features to protect a site: brute force protection, security scanning, and a WAF. RNA labels Feb 25, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 25, 2026

Thank you for your PR!

When contributing to Jetpack, we have a few suggestions that can help us test and review your patch:

  • ✅ Include a description of your PR changes.
  • ✅ Add a "[Status]" label (In Progress, Needs Review, ...).
  • ✅ Add testing instructions.
  • ✅ Specify whether this PR includes any changes to data or privacy.
  • ✅ Add changelog entries to affected projects

This comment will be updated as you work on your PR and make changes. If you think that some of those checks are not needed for your PR, please explain why you think so. Thanks for cooperation 🤖


Follow this PR Review Process:

  1. Ensure all required checks appearing at the bottom of this PR are passing.
  2. Make sure to test your changes on all platforms that it applies to. You're responsible for the quality of the code you ship.
  3. You can use GitHub's Reviewers functionality to request a review.
  4. When it's reviewed and merged, you will be pinged in Slack to deploy the changes to WordPress.com simple once the build is done.

If you have questions about anything, reach out in #jetpack-developers for guidance!


Boost plugin:

No scheduled milestone found for this plugin.

If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack.


Protect plugin:

No scheduled milestone found for this plugin.

If you have any questions about the release process, please ask in the #jetpack-releases channel on Slack.

@github-actions github-actions bot added [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. and removed [Status] Needs Review This PR is ready for review. labels Feb 25, 2026
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 pull request removes Jetpack color overrides on core WordPress components across multiple plugins and packages, allowing components to use default WordPress core styling. This is part of a larger effort (spin-off from PR #47056) to reduce custom styling and improve consistency with WordPress core.

Changes:

  • Removed --wp-admin-theme-color* CSS custom property overrides from multiple plugin and package stylesheets
  • Removed custom toggle styling to rely on WordPress core toggle components
  • Removed custom button styling including border-radius overrides and secondary button styles
  • Removed unused --jp-highlight CSS variable from root variables

Reviewed changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
projects/plugins/protect/src/js/styles.module.scss Removed color theme overrides and focus border width customization
projects/plugins/boost/app/assets/src/css/main/wp-admin.scss Removed color theme overrides and secondary button styling (has issue with remaining variable reference)
projects/plugins/boost/app/assets/src/css/admin-banner.scss Removed custom CTA button styling (background, border, color, hover, focus states)
projects/packages/videopress/src/client/admin/components/video-thumbnail/style.module.scss Removed spinner color override
projects/packages/search/src/dashboard/components/module-control/style.scss Removed custom toggle styling and related variables
projects/packages/publicize/_inc/components/admin-page/toggles/styles.module.scss Removed toggle color override
projects/packages/publicize/_inc/components/admin-page/toggles/social-notes-toggle/styles.module.scss Removed toggle color override
projects/packages/my-jetpack/_inc/style.module.scss Removed color theme overrides
projects/js-packages/components/components/toggle-control/styles.module.scss Removed toggle color override
projects/js-packages/components/components/button/style.module.scss Removed color theme overrides and border-radius customization
projects/js-packages/base-styles/root-variables.scss Removed unused --jp-highlight variable

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

@@ -11,11 +11,6 @@ p {
}
// Override WordPress component styles
.jb-dashboard {
Copy link

Copilot AI Feb 25, 2026

Choose a reason for hiding this comment

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

The custom override for --wp-admin-theme-link-color has been removed, but this variable is still referenced on lines 36 and 40 of this file. Since the custom override was #{variables.$gray_90}, removing it means these elements will now use WordPress core's default link color instead of the previous gray color. This could result in a visual change. If this is intentional, the references should ideally be updated to either use the core variable directly or be removed if the default behavior is desired.

Copilot uses AI. Check for mistakes.
Copy link
Member Author

Choose a reason for hiding this comment

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

They might still be needed, but needs a closer look:

a {
color: var(--wp-admin-theme-link-color);
}
.components-button.is-link:not(:disabled) {
color: var(--wp-admin-theme-link-color);
}

These also are related and need a closer look to see what they actually affect:

// Override WP styles for panel header.
.components-panel__body-title .components-button:hover:not(:disabled,[aria-disabled="true"]),
.components-panel__body-title .components-button[aria-expanded="true"] {
color: inherit;
}
.components-panel__body-toggle.components-button:focus {
border-radius: inherit;
box-shadow: none;
}

@simison simison added [Status] In Progress and removed [Status] Needs Author Reply We need more details from you. This label will be auto-added until the PR meets all requirements. labels Feb 25, 2026
@jp-launch-control
Copy link

Code Coverage Summary

This PR did not change code coverage!

That could be good or bad, depending on the situation. Everything covered before, and still is? Great! Nothing was covered before? Not so great. 🤷

Full summary · PHP report · JS report

Comment on lines -79 to -87
color: variables.$primary-black;
background: none;
border: 1px solid variables.$primary-black;
padding: 8px 24px;
border-radius: 3px;
font-weight: 600;
cursor: pointer;
font-size: 16px;
line-height: 24px;
Copy link
Member Author

Choose a reason for hiding this comment

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

Some non-color changes here so need to double check where it was applied in Boost and check properly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Feature] Publicize Now Jetpack Social, auto-sharing [JS Package] Base Styles [JS Package] Components [Package] My Jetpack [Package] Publicize [Package] Search Contains core Search functionality for Jetpack and Search plugins [Package] VideoPress [Plugin] Boost A feature to speed up the site and improve performance. [Plugin] Protect A plugin with features to protect a site: brute force protection, security scanning, and a WAF. RNA [Status] In Progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant