Problem/Motivation

When Adds reset facets link configuration is set to true in facet-summary configuration, there is no link visible on the facet summary block.

Steps to reproduce

1. Create a facet summary for by clicking on + Add facet summary
2. Then enable the configuration mention like above.
3. Add that summary block into any region (ex: Facet region).
4. Then select some facet items on product list page.
5. You will see the selected product in facet summary block but Reset All link will not be visible.

CommentFileSizeAuthor
#2 3173972-2.patch1.68 KBtimotej-pl

Issue fork facets-3173972

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

abhilash muniganti created an issue. See original summary.

timotej-pl’s picture

timotej-pl CreditAttribution: timotej-pl at Agiledrop - Your Trusted Drupal Teammates commented
Status: Active » Needs review
FileSize
3173972-2.patch1.68 KB

I tried to reproduce this issue, but on my environment it works just fine. The only thing is that you got to make sure that 'Reset facets link text' value under 'Adds reset facets link settings' is set, because if not, the reset link won't be displayed.
I created a patch that ignores that, if for some reason you don't want to set text in settings. It sets a default 'Reset' value to reset link.

borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ at Calibrate commented
Status: Needs review » Reviewed & tested by the community

Looks great, thanks @timotej-pl!

  • borisson_ committed 77b3014 on 8.x-1.x authored by timotej-pl 
    Issue #3173972 by timotej-pl: Facet summary Reset all link is not...
borisson_’s picture

borisson_
Primary language Dutch
Location Mechelen, 🇧🇪
CreditAttribution: borisson_ at Calibrate commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

Amine Allagui made their first commit to this issue’s fork.