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

Add HTMX at search #45

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

newtoallofthis123
Copy link

@newtoallofthis123 newtoallofthis123 commented Nov 6, 2023

Regarding #44

This commit adds htmx to the main search bar at the home page, just for a test....
this is how it looks:

2023-11-06_19-04-10.mp4

However, here are some things I had to do to get this and some optimizations:

  1. I added a main search route and two subsidiaries to it called /search, for normal search and /form_search that only works with post requests
  2. Had to add a search_partial.tmpl that only had the styled div of the search results
  3. {MAIN} I had to remove the default-src 'self'; script-src 'self'; from the CSP... I am not sure where the CSP is being used, so I am sorry if it was neccessay, please let me know

Other than that, hopefully, this is a nice demo :)

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.

1 participant