woocommerce-blocks icon indicating copy to clipboard operation
woocommerce-blocks copied to clipboard

Slider of Price filter block should use currentColor instead of a hard-coded color

Open Aljullu opened this issue 3 years ago • 4 comments
trafficstars

Describe the bug

When the Price filter block is added on a dark background, the slider can't be seen.

To reproduce

  1. Add the Filter by Price and All Products blocks inside a Columns block into a post or page.
  2. Change the background color of the Filter by Price column to a dark color. Change the text color to a light color.
  3. Notice the price slider is no longer visible.

Expected behavior

That's because we have some hard-coded styles in the slider. Ideally, we should default to currentColor.

Screenshots

Aljullu avatar Sep 13 '22 07:09 Aljullu

Could I contribute to this issue?

EmptySet-Exe avatar Sep 13 '22 22:09 EmptySet-Exe

Sure, go for it, @EmptySet-Exe! You will probably need to look into this CSS file: woocommerce-blocks/assets/js/base/components/price-slider/style.scss.

You can read how to build the project and contribute to it in the Getting Started guide. Let us know if you have any questions and thanks for your effort!

Aljullu avatar Sep 14 '22 06:09 Aljullu

Thank you! Will I have to reach out for any questions here or...?

EmptySet-Exe avatar Sep 20 '22 11:09 EmptySet-Exe

Yes, you can posts your questions here and we will do our best to answer all your questions. :slightly_smiling_face:

Aljullu avatar Sep 20 '22 12:09 Aljullu

Hey I was wondering if you can help me with a hosting issue that I have?

EmptySet-Exe avatar Oct 02 '22 18:10 EmptySet-Exe

Hey @EmptySet-Exe, feel free to drop any question that you have here :)

gigitux avatar Oct 03 '22 08:10 gigitux

Hi, I'm having trouble viewing my changes in the style.scss show up in my WordPress instance.

EmptySet-Exe avatar Oct 03 '22 08:10 EmptySet-Exe

My suggestion is to set up a local website to work on this issue. You could use LocalWP.

gigitux avatar Oct 03 '22 08:10 gigitux

Hey I've noticed that the WooCommerce Blocks don't appear in the WP Editor unless if I have the WooCommerce Plugin installed. That's to say that the WooCommerce Blocks don't show up in the Block Inserter if I only have the WooCommerce Block Plugin installed alone. Is that how it's supposed to be, because I can't ensure that my changes are directly appearing on the webpage otherwise.

EmptySet-Exe avatar Oct 03 '22 08:10 EmptySet-Exe

Hey I've noticed that the WooCommerce Blocks don't appear in the WP Editor unless if I have the WooCommerce Plugin installed. That's to say that the WooCommerce Blocks don't show up in the Block Inserter if I only have the WooCommerce Block Plugin installed alone. Is that how it's supposed to be, because I can't ensure that my changes are directly appearing on the webpage otherwise.

Follow these steps for configuring your local env.

gigitux avatar Oct 03 '22 08:10 gigitux

Excuse me, I wanted to know if there are any guidelines for crafting a pull request?

EmptySet-Exe avatar Oct 03 '22 10:10 EmptySet-Exe

Excuse me, I wanted to know if there are any guidelines for crafting a pull request?

In this folder, there are all the guidelines for the contributors. When you open the PR, you will see a template, just fill the sections and open the PR :)

gigitux avatar Oct 03 '22 10:10 gigitux

Which branch should I make the PR from for this?

EmptySet-Exe avatar Oct 03 '22 19:10 EmptySet-Exe

Hi! It seems that you created the PR https://github.com/woocommerce/woocommerce-blocks/pull/7278. Thanks for working on this! I will take a look in the next few days!

gigitux avatar Oct 04 '22 14:10 gigitux

Hey, just curious, can you give me an idea as to why my contribution had failed some of those tests?

EmptySet-Exe avatar Oct 08 '22 07:10 EmptySet-Exe

Hi @EmptySet-Exe! We have some problems with our current E2E tests, so don't worry about that. Instead, could you fix the CSS linting issue? (https://github.com/woocommerce/woocommerce-blocks/pull/7278#issuecomment-1267115947)

gigitux avatar Oct 10 '22 07:10 gigitux

Okay, I'll take a look at it. Thanks!

EmptySet-Exe avatar Oct 10 '22 18:10 EmptySet-Exe

Thank you for allowing me to contribute!

EmptySet-Exe avatar Nov 04 '22 21:11 EmptySet-Exe