From 246a459622705e04f93a5cbac24c8d26dd6f9081 Mon Sep 17 00:00:00 2001
From: Jim Myhrberg <contact@jimeh.me>
Date: Thu, 10 Mar 2022 21:11:52 +0000
Subject: [PATCH] fix(navigation): remove ctrlf package in favor of isearch

Visually ctrlf is very nice, but it seems one of it's features is to
stay active in the mini-buffer until explicitly cancelled. This can
easily lead to annoyance if you mess up a C-x C-s keybind and then move
on to other buffers and such, as ctrlf will still be active.

Regular isearch however will dismiss itself as soon as you do anything
other than typing in a search query. Hence causing a lot less annoyance
for me.
---
 core/siren-core-modules.el          |  2 +-
 modules/navigation/siren-isearch.el | 18 ++++++++++++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)
 create mode 100644 modules/navigation/siren-isearch.el

diff --git a/core/siren-core-modules.el b/core/siren-core-modules.el
index 6c88150..3281dfe 100644
--- a/core/siren-core-modules.el
+++ b/core/siren-core-modules.el
@@ -65,7 +65,6 @@
 (require 'siren-avy)
 (require 'siren-centaur-tabs)
 (require 'siren-consult)
-(require 'siren-ctrlf)
 (require 'siren-dired)
 (require 'siren-dumb-jump)
 (require 'siren-git-link)
@@ -73,6 +72,7 @@
 (require 'siren-helm-ag)
 (require 'siren-helm-swoop)
 (require 'siren-imenu)
+(require 'siren-isearch)
 (require 'siren-recursive-narrow)
 (require 'siren-scroll-half-screen)
 
diff --git a/modules/navigation/siren-isearch.el b/modules/navigation/siren-isearch.el
new file mode 100644
index 0000000..82903e5
--- /dev/null
+++ b/modules/navigation/siren-isearch.el
@@ -0,0 +1,18 @@
+;;; siren-isearch.el --- jimeh's Emacs Siren: isearch configuration.
+
+;;; Commentary:
+
+;; Basic configuration for isearch.
+
+;;; Code:
+
+(use-package isearch
+  :straight (:type built-in)
+  :custom
+  (isearch-lazy-count t)
+  (regexp-search-ring-max 200)
+  (search-default-mode t)
+  (search-ring-max 200))
+
+(provide 'siren-isearch)
+;;; siren-isearch.el ends here