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

Move facets to footer. #6

Merged
merged 4 commits into from
Jan 28, 2022
Merged

Move facets to footer. #6

merged 4 commits into from
Jan 28, 2022

Conversation

rosiel
Copy link
Contributor

@rosiel rosiel commented Jan 12, 2022

Move facet info to the footer of the block. Sorry this took me so long, I tried a million ways that didn't work to get these to show up in the footer. If you have a working site with demo content and the install profile, testing should be as easy as loading this YML through config sync and seeing that the facets start showing at the bottom instead of the top, whenever a search view is rendered.

I don't know that using the "header" and "footer" view regions is the best long term solution for placing facet blocks. I do know that doing it this way makes it a little less theme dependent.

@Natkeeran @kstapelfeldt

@Natkeeran
Copy link
Contributor

@rosiel

I manually overwrote it by the view config by going to https://islandora.traefik.me/admin/config/development/configuration/full/import. Then cache cleared and rebuild the index as well.

The general search works, but the alignment is off. I've attached a screen shot.
footer_facets

https://islandora.traefik.me/solr-search/content?search_api_fulltext=&sort_by=search_api_relevance&sort_order=ASC&items_per_page=10

However, when navigating to the collections, it does not display the facets as one would expect.

@noahwsmith
Copy link
Contributor

Danny, Nat and I looked at this and it looks good except that the same changes need to be made to this section as well

@Natkeeran Natkeeran merged commit afa7da9 into main Jan 28, 2022
@rosiel rosiel deleted the fix-collection-views branch January 31, 2022 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants