Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Experimental] Price Filter: Fix styling issue when Inline input fields is enabled #45197

Conversation

thealexandrelara
Copy link
Contributor

@thealexandrelara thealexandrelara commented Feb 28, 2024

Submission Review Guidelines:

Changes proposed in this Pull Request:

Closes #44755.

How to test the changes in this Pull Request:

Using the WooCommerce Testing Instructions Guide, include your detailed testing instructions:

  1. Navigate to Single Product template (Appearance > Editor > Templates > Product Catalog template - or any archive template).
  2. Click on "Edit" to open the WordPress editor.
  3. In the editor, you should see a block inserter icon or an option to "Add Block". Click on it.
  4. In the block inserter, search for "Product Filter" or browse the available blocks until you find them.
  5. Click on the "Product Filter: Price (Beta)" block to add it to your content.
  6. Select the Product Filter: Price (Beta) block in the blocks list.
  7. In the sidebar to the right side, enable the "Inline input fields" setting within the Inspector Controls.
  8. Make sure the block is displayed with the input fields at the sides of the price range slider.
  9. Click on Save to save the changes.
  10. Visit the Shop page (https://linktoyourstore.com/shop/).
  11. Make sure the block is displayed with the input fields at the sides of the price range slider.
Before After
CleanShot 2024-02-29 at 18 22 34@2x CleanShot 2024-03-05 at 10 40 52@2x

Changelog entry

  • Automatically create a changelog entry from the details below.

Significance

  • Patch
  • Minor
  • Major

Type

  • Fix - Fixes an existing bug
  • Add - Adds functionality
  • Update - Update existing functionality
  • Dev - Development related task
  • Tweak - A minor adjustment to the codebase
  • Performance - Address performance issues
  • Enhancement - Improvement to existing functionality

Message

Fix styling issue for the Price Filter block preventing fields from appearing inline when the Inline input fields option is enabled

Comment

@thealexandrelara thealexandrelara added type: bug The issue is a confirmed bug. team: Kirigami & Origami block: filter by price Issues related to the Filter by Price block. labels Feb 28, 2024
@thealexandrelara thealexandrelara self-assigned this Feb 28, 2024
Copy link
Contributor

github-actions bot commented Feb 28, 2024

Test Results Summary

Commit SHA: 08453ad

Test 🧪Passed ✅Failed 🚨Broken 🚧Skipped ⏭️Unknown ❔Total 📊Duration ⏱️
API Tests25900202610m 38s
E2E Tests338001303518m 23s

To view the full API test report, click here.
To view the full E2E test report, click here.
To view all test reports, visit the WooCommerce Test Reports Dashboard.

@github-actions github-actions bot added the plugin: woocommerce Issues related to the WooCommerce Core plugin. label Feb 29, 2024
@thealexandrelara thealexandrelara marked this pull request as ready for review February 29, 2024 21:48
@woocommercebot woocommercebot requested review from a team and kmanijak and removed request for a team February 29, 2024 21:48
Copy link
Contributor

Hi @kmanijak,

Apart from reviewing the code changes, please make sure to review the testing instructions as well.

You can follow this guide to find out what good testing instructions should look like:
https://github.com/woocommerce/woocommerce/wiki/Writing-high-quality-testing-instructions

Copy link
Contributor

@kmanijak kmanijak left a comment

Choose a reason for hiding this comment

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

Hi @thealexandrelara,

I found the testing instructions a bit confusing 😅

  1. In the editor, you should see a block inserter icon or an option to "Add Block". Click on it.
  2. In the block inserter, search for "Filter by Price" or browse the available blocks until you find them.
  3. Click on the "Filter by Price" block to add it to your content.
  4. Select the Product Filter: Price (Beta) block in the blocks list.

This is a pretty confusing part to me. I think it could merged into a single point telling which block exactly to insert. I checked both "Filter by Price" and "Product Filter: Price (Beta)" as I wasn't sure.

In the sidebar to the left side, enable the "Inline input fields" setting.

I assume you mean Inspector Controls, then they're on the right.


Testing:

  1. I tested the changes and the inline layout works just fine. I'm just curious if the currency sign is expected to be there as on AFTER image? In my case, it's not there either in the Editor or frontend.

  2. Also, I know it's not part of this PR but is it a known issue that when you input some value manually in the Product Filter: Price (Beta) the filter doesn't react to that change? Blurring the input and clicking ENTER doesn't help.

pricefilter.mov

Let me know if these two issues are expected and if so - I'll go ahead and approve the PR.

@thealexandrelara
Copy link
Contributor Author

Hi @thealexandrelara,

I found the testing instructions a bit confusing 😅

Thank you for pointing this out, I believe my mind went for a walk when I started writing the testing instructions 😅. I just updated it to make it more clear. The correct block is the Product Filter: Price (Beta), and yes, you're correct I'm referring to the Inspector Controls to the right.

  1. I tested the changes and the inline layout works just fine. I'm just curious if the currency sign is expected to be there as on AFTER image? In my case, it's not there either in the Editor or frontend.

That is really well noted! I just opened an issue for this #45315

  1. Also, I know it's not part of this PR but is it a known issue that when you input some value manually in the Product Filter: Price (Beta) the filter doesn't react to that change? Blurring the input and clicking ENTER doesn't help.

pricefilter.mov
Let me know if these two issues are expected and if so - I'll go ahead and approve the PR.

This is a known issue, there is an issue created for this #45228. Hopefully, I'll be able to work on this today 🙏

Copy link
Contributor

@kmanijak kmanijak left a comment

Choose a reason for hiding this comment

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

Thanks for addressing my concerns! As there are issues covering the remaining parts I'm approving this one! 🚀

@thealexandrelara thealexandrelara merged commit 8163b83 into trunk Mar 6, 2024
61 checks passed
@thealexandrelara thealexandrelara deleted the fix/44755-price-filter-styling-issue-when-inline-input-fields-is-enabled branch March 6, 2024 20:10
@github-actions github-actions bot added this to the 8.8.0 milestone Mar 6, 2024
@github-actions github-actions bot added the needs: analysis Indicates if the PR requires a PR testing scrub session. label Mar 6, 2024
@Stojdza Stojdza added needs: external testing Indicates if the PR requires further testing conducted by testers external to the development team. status: analysis complete Indicates if a PR has been analysed by Solaris and removed needs: analysis Indicates if the PR requires a PR testing scrub session. labels Mar 6, 2024
@dinhtungdu dinhtungdu changed the title Price Filter: Fix styling issue when Inline input fields is enabled [Experimental] Price Filter: Fix styling issue when Inline input fields is enabled Mar 7, 2024
Konamiman pushed a commit that referenced this pull request Mar 13, 2024
…45197)

* Fix styling issue in the Editor when Inline input fields is enabled

* Add changefile(s) from automation for the following project(s): woocommerce-blocks

* Fix styling issue on the Frontend when Inline input fields is enabled

* Fix css lint errors

---------

Co-authored-by: github-actions <github-actions@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
block: filter by price Issues related to the Filter by Price block. needs: external testing Indicates if the PR requires further testing conducted by testers external to the development team. plugin: woocommerce Issues related to the WooCommerce Core plugin. status: analysis complete Indicates if a PR has been analysed by Solaris team: Kirigami & Origami type: bug The issue is a confirmed bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Price Filter styling issue when Inline Input Field is enabled
3 participants