Exposed filters: Support views_filters_summary

Created on 11 December 2023, 7 months ago
Updated 14 May 2024, about 1 month ago

When using facets as exposed filters, we should use views_filters_summary instead of our own solution.

Needs work

✨ Feature request
Status

Needs work

Version

3.0

Component

Code

Created by

🇧🇪Belgium StryKaizer Belgium

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

  • Issue created by @StryKaizer
  • 🇧🇪Belgium StryKaizer Belgium
  • 🇧🇪Belgium StryKaizer Belgium
  • 🇧🇪Belgium StryKaizer Belgium

    Patch attached is also required, this is a patch for the views_filters_summary module.

    Once this has been tested, I'll create an issue in the views_filters_summary queue...

  • First commit to issue fork.
  • 🇮🇳India Shyam16102001

    Shyam16102001 → changed the visibility of the branch 3408001-exposed-filters-support to hidden.

  • Status changed to Needs work about 1 month ago
  • 🇺🇸United States danflanagan8 St. Louis, US

    I applied both patches. I keep seeing this warning:

    Warning: foreach() argument must be of type array|object, string given in Drupal\facets_exposed_filters\Plugin\views\filter\FacetsFilter->acceptExposedInput() (line 141 of modules/contrib/facets/modules/facets_exposed_filters/src/Plugin/views/filter/FacetsFilter.php).

    The text of the pills looks good, but the X links don't seem to be working for me. I'm new to views_filters_summary so I don't know if that is necessarily related to facets.

  • 🇺🇸United States brayn7 Lexington, Ky

    This doesn't apply for me with drupal 10.2. Additionally, and I should probably create a ticket for it, but removing a facet selection via the summary just resets all the filters because it is just a link to the view page im currently on. In ViewsExposedFilters.php there is a comment "We don't build real urls here. Views will handle this part for us." But thats not happening. Could be a conflict with ajax maybe idk.

Production build 0.69.0 2024