Skip to content

Conversation

yogeshbhutkar
Copy link
Contributor

What?

Part of #69646

This pull request adds theme support to the ToolsPanel component.

Why?

The component has visibility issues in Dark Themes.

How?

  1. Deprecated variables such as COLORS.gray[ 700 ] are replaced with COLORS.theme.gray[ 700 ]
  2. Theme-based background and foreground are added to the component

Testing Instructions

  1. Run storybook locally. (npm run storybook:dev)
  2. Navigate to the ToolsPanel component.
  3. Change the theme and confirm that there are no visibility issues.

Testing Instructions for Keyboard

Same

Screenshots

Before After
before after

@@ -1,6 +1,6 @@
.components-panel {
background: $white;
border: $border-width solid $gray-200;
border: $border-width solid $components-color-gray-200;
Copy link
Contributor Author

Choose a reason for hiding this comment

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

The Panel component still needs a full migration to themes, but I wanted to keep this PR focused on ToolsPanel. That said, I had to update this line because ToolsPanel inherits it.

@yogeshbhutkar yogeshbhutkar marked this pull request as ready for review April 4, 2025 06:22
@yogeshbhutkar yogeshbhutkar requested a review from ajitbohra as a code owner April 4, 2025 06:22
Copy link

github-actions bot commented Apr 4, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: yogeshbhutkar <yogeshbhutkar@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@t-hamano t-hamano added [Type] Enhancement A suggestion for improvement. [Package] Components /packages/components labels Apr 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Package] Components /packages/components [Type] Enhancement A suggestion for improvement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants