diff --git a/core/autoload/projects.el b/core/autoload/projects.el index 0af779171..57d4c8b01 100644 --- a/core/autoload/projects.el +++ b/core/autoload/projects.el @@ -128,6 +128,8 @@ If DIR is not a project, it will be indexed (but not cached)." (if (doom-module-p :completion 'ivy) #'counsel-projectile-find-file #'projectile-find-file))) + ((fboundp 'selectrum-mode) ;HACK see @ymarco's comment on #5013 and TODO.org in the selecturm module. + (call-interactively #'find-file)) ((fboundp 'counsel-file-jump) ; ivy only (call-interactively #'counsel-file-jump)) ((project-current nil dir) diff --git a/modules/completion/selectrum/TODO.org b/modules/completion/selectrum/TODO.org index 966b4bc8b..89787c607 100644 --- a/modules/completion/selectrum/TODO.org +++ b/modules/completion/selectrum/TODO.org @@ -10,6 +10,16 @@ regular isearch for now. ** TODO =SPC s B= Selectrum/Consult don't have a ~swiper-all~ analogue either. Unbound for now. +* PROJ HACKs to be addressed +** ~fboundp~ issues +Even if the =ivy= module isn't loaded, it's packages can still get loaded by +other means, such as =lispy= requiring =counsel=. This means that the ~fboundp~ +logic such [[file:~/.emacs.d/modules/config/default/autoload/text.el::(cond ((fboundp 'consult-yank-pop) #'consult-yank-pop) ;;HACK see @ymarco's comment on #5013 and TODO.org][here]] and [[file:~/.emacs.d/core/autoload/projects.el::((fboundp 'selectrum-mode) ;HACK see @ymarco's comment on #5013 and TODO.org][here]] won't work unless the selectrum option is checked +first, which is what we do for now. +** ~projectile-switch-project-action~ definition +Without [[file:~/.emacs.d/modules/ui/workspaces/config.el::;; HACK?? needs review][this]] change new projects don't get opened in a new tab, but the exact +working of this whole set up are a bit opaque to me. + * PROJ Bugs ** TODO =C-SPC= and live previews Automatic live previews have been disabled on slow ~consult~ commands. diff --git a/modules/config/default/autoload/text.el b/modules/config/default/autoload/text.el index b70848c3a..5da7f4937 100644 --- a/modules/config/default/autoload/text.el +++ b/modules/config/default/autoload/text.el @@ -27,8 +27,8 @@ "Interactively select what text to insert from the kill ring." (interactive) (call-interactively - (cond ((fboundp 'counsel-yank-pop) #'counsel-yank-pop) - ((fboundp 'consult-yank-pop) #'consult-yank-pop) + (cond ((fboundp 'consult-yank-pop) #'consult-yank-pop) ;HACK see @ymarco's comment on #5013 and TODO.org in the selecturm module. + ((fboundp 'counsel-yank-pop) #'counsel-yank-pop) ((fboundp 'helm-show-kill-ring) #'helm-show-kill-ring) ((error "No kill-ring search backend available. Enable ivy, helm or selectrum!")))))