Question: jQuery requirement

[thomasjsn]

Hey, what addition in the latest release requires jQuery? I've managed to make due without it, and hoping to keep it that way.

[mmistakes]

It's there for search. If you're not using that you can safely remove it.

[thomasjsn]

I am currently using Algolia for search on my site, and would like to use the search icon and search page that the latest release contains. Would I need jQuery for that?

[mmistakes]

Yes you'll need jQuery, unless you want to rewrite this search toggle script in vanilla JS.

// Search toggle
$(".search-toggle").on("click", function() {
  $(".search-content").toggleClass("is--visible");
  $(".initial-content").toggleClass("is--hidden");
  // set focus on input
  setTimeout(function() {
    $("#search").focus();
  }, 400);
});

I'll probably get to redoing it at some point to remove the extra dependency. Happy to merge a PR if someone wants to tackle it for me.

[thomasjsn]

I see, thanks for the reply. I think I'll just use /search as I do now, with the search icon as a link and the new search results html you added :)

[thomasjsn]

I've looked a bit more that this, wouldn't this be enough; https://stackoverflow.com/a/19075558? I'll be experimenting a bit with it...

[mmistakes]

Yeah it shouldn't be too difficult. I'm already doing something similar.

I pulled most of the logic from one of my other themes, which has the jQuery dependency. I just didn't have time to go in and rewrite things to be vanilla for this theme.

[thomasjsn]

I see, thanks for the feedback. Your theme and writing about Jekyll saved me so much time making my site, thank you.

[mmistakes]

Fixed in 7716577