Skip to content

Add search to sidebar

username-removed-25042 requested to merge pserwylo/fdroid-website:search into master

Search feature which was implemented in jekyll-fdroid now implemented in the website.

This doesn't need to bump the jekyll-fdroid dependency, that was already done in !78 (merged) because there was a non-search-related fix in there.

If JavaScript is disabled, I would like some feedback on whether it should show:

  • Find Apps "Requires JavaScript to be enabled" (as implemented in this MR)
  • "Finding apps requires JavaScript to be enabled" (i.e. without the large heading)
  • Don't even show the sidebar at all

Thoughts?

https://search-fdroid-website-pserwylo.surge.sh/fdroid-website

Fixes #76 (closed).

Edited by username-removed-25042

Merge request reports