fix(irc): circe-color-nicks & circe-new-day-notifier

Prior to this, the user would need to know the precise load order of
circe and circe-color-nicks/circe-new-day-notifier to overwrite Doom's
defaults in an `after!` or `with-eval-after-load` block.

Also, `enable-circe-color-nicks` does not have buffer-local
side-effects, so shouldn't be hooked to `circe-channel-mode-hook`.

Close: #7959
Co-authored-by: Thaodan <Thaodan@users.noreply.github.com>
This commit is contained in:
Henrik Lissner 2024-07-23 14:00:40 -04:00
parent c95ecc6293
commit 6d7e3ad365
No known key found for this signature in database
GPG key ID: B60957CA074D39A3

View file

@ -132,14 +132,21 @@ playback.")
(use-package! circe-color-nicks (use-package! circe-color-nicks
:hook (circe-channel-mode . enable-circe-color-nicks) :defer t
;; NOTE: I avoid `:after' on purpose, because it convolutes load order and
;; makes it harder for users to know what to target with `after!' or
;; `with-eval-after-load' when trying to configure packages.
:init (after! circe (require 'circe-color-nicks))
:config :config
(setq circe-color-nicks-min-constrast-ratio 4.5 (setq circe-color-nicks-min-constrast-ratio 4.5
circe-color-nicks-everywhere t)) circe-color-nicks-everywhere t)
(enable-circe-color-nicks))
(use-package! circe-new-day-notifier (use-package! circe-new-day-notifier
:after circe :defer t
;; NOTE: See NOTE in `circe-color-nicks' above.
:init (after! circe (require 'circe-new-day-notifier))
:config :config
(enable-circe-new-day-notifier) (enable-circe-new-day-notifier)
(setq circe-new-day-notifier-format-message (setq circe-new-day-notifier-format-message