Support for Facet API Pretty Paths

Created on 21 August 2013, almost 11 years ago
Updated 11 March 2024, 4 months ago

Hello,

I've added support for Facet API Pretty Paths to AJAX Facets. Well, at least to my version of AJAX Facets. The attached patch adds pretty paths support but also includes the patch from #2066195: Update Facets on Exposed Filter submission β†’ . Sorry, this is not really the best way to submit patches, but I figure it's better to post something and get a dialog going then do nothing. Also, maybe if someone else is looking for Exposed filter and Pretty paths support they'll be able to test/contribute somehow.

Anyway, I consider this patch more of a proof of concept. If you like the direction I'm heading with it, maybe we can work on it further? Or you can just take the reins if you are so inclined. Let me know what you think!

Thanks!
drclaw

✨ Feature request
Status

Needs review

Version

3.0

Component

Code

Created by

πŸ‡¨πŸ‡¦Canada drclaw

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

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΈπŸ‡°Slovakia gresko8

    Here is my initial attempt for 7.x-3.x patch. It might need some improvement in a sense that we could maybe get rid of some code execution when using the pretty paths but it seems to be working fine for me.

    The patch deals with checkboxes and links compatibility with pretty paths. I didn't have time to work on select/ranges widgets.

    I tested multiple options per facet and navigation through browser history as well.

  • πŸ‡ΈπŸ‡°Slovakia gresko8

    Attaching an updated patch which also includes search query in history.

  • πŸ‡ΈπŸ‡°Slovakia gresko8

    One more fix to unset the pretty_path query param so it doesn't clutter up views links.

  • πŸ‡ΈπŸ‡°Slovakia gresko8

    I found out there were some warnings logged and setting breadcrumbs did not work correctly. Attaching a patch to fix that by making sure we are using the system path and not an alias in the background.

Production build 0.69.0 2024