Skip to content
This repository has been archived by the owner on Feb 23, 2024. It is now read-only.

Product categories list block hierarchy display fix #4920

Merged
merged 1 commit into from
Oct 12, 2021

Conversation

ovidiul
Copy link
Contributor

@ovidiul ovidiul commented Oct 8, 2021

This PR fixes #4625 product categories list block display by HTML encoding the minus sign, which is the hierarchy prefix.

Fixes #4625

Accessibility

Screenshots

Screenshot 2021-10-07 at 14 29 24

Testing

Automated Tests

  • Changes in this PR are covered by Automated Tests.
    • Unit tests
    • E2E tests

Manual Testing

How to test the changes in this Pull Request:

  1. Create 4 WooCommerce product categories, each one nested under the previous one created, like follows:
  • Level 1
    • Level 2
      • Level 3
        • Level 4
  1. Add a product to the last category
  2. To a page, add the block Product Categories List
  3. Under the block settings > List settings > Display style, pick "Dropdown"
  4. Again under the block settings > Content, switch On "Show hierarchy"
  5. Save and view page

Performance Impact

Changelog

Fix the dropdown list in Product Category List Block for nested categories

@ovidiul ovidiul marked this pull request as ready for review October 8, 2021 18:20
@ralucaStan ralucaStan self-requested a review October 11, 2021 16:49
@ralucaStan
Copy link
Contributor

I've tested this PR and it works as expected.
However, I see a linting step failing on the CI. Would you mind bringing this branch up to date with trunk (rebase trunk)?

@ovidiul
Copy link
Contributor Author

ovidiul commented Oct 12, 2021

I've tested this PR and it works as expected. However, I see a linting step failing on the CI. Would you mind bringing this branch up to date with trunk (rebase trunk)?

Sure, its done now, however I think its failing for external reasons to the code, seeing this at the Linting step:

Run ataylorme/[email protected]
Error: Resource not accessible by integration

@ovidiul
Copy link
Contributor Author

ovidiul commented Oct 12, 2021

Seems I've added another non-related commit in there, fixing that in a moment.

@ovidiul ovidiul closed this Oct 12, 2021
@ovidiul ovidiul force-pushed the fix_product_cat_list_display branch from 9a4d01a to c9570c0 Compare October 12, 2021 09:56
@ovidiul ovidiul reopened this Oct 12, 2021
@ovidiul ovidiul force-pushed the fix_product_cat_list_display branch from da2ff4d to 397a908 Compare October 12, 2021 10:02
@ovidiul
Copy link
Contributor Author

ovidiul commented Oct 12, 2021

Non-related commit fixed, as per my previous reply, JavaScript Linting fails with:

Run ataylorme/[email protected]
Error: Resource not accessible by integration

@ralucaStan
Copy link
Contributor

Merging this as the failed JavaScript and CSS Linting / Lint JavaScript step is the result of a known limitation of Github actions.
@ovidiul thank you for fixing this.

@ralucaStan ralucaStan merged commit b7574b2 into woocommerce:trunk Oct 12, 2021
opr added a commit that referenced this pull request Oct 12, 2021
* Empty commit for release pull request

* Update readme.txt with release changelog

* Add testing instructions for 6.1.0 release

* Update readme.txt with release changelog

* #4625 product categories list block hierarchy display fix (#4920)

* Update link to testing zip

* Remove mini cart testing from instructions

This is because mini-cart is only in experimental builds.

* Update changelog and testing notes

* Correct typo in testing instructions

* Update testing instructions

* Bumping version strings to new version.

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Thomas Roberts <[email protected]>
Co-authored-by: Ovidiu Liuta <[email protected]>
@opr opr added this to the 6.1.0 milestone Oct 25, 2021
jonny-bull pushed a commit to jonny-bull/woocommerce-gutenberg-products-block that referenced this pull request Dec 14, 2021
jonny-bull pushed a commit to jonny-bull/woocommerce-gutenberg-products-block that referenced this pull request Dec 14, 2021
* Empty commit for release pull request

* Update readme.txt with release changelog

* Add testing instructions for 6.1.0 release

* Update readme.txt with release changelog

* woocommerce#4625 product categories list block hierarchy display fix (woocommerce#4920)

* Update link to testing zip

* Remove mini cart testing from instructions

This is because mini-cart is only in experimental builds.

* Update changelog and testing notes

* Correct typo in testing instructions

* Update testing instructions

* Bumping version strings to new version.

Co-authored-by: github-actions <[email protected]>
Co-authored-by: Thomas Roberts <[email protected]>
Co-authored-by: Ovidiu Liuta <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Woo Product Categories List not visually differentiating between Level 3 and Level 4 categories
3 participants