diff --git a/TODO.org b/TODO.org
index 79ff6f0bf..26450d381 100644
--- a/TODO.org
+++ b/TODO.org
@@ -105,6 +105,7 @@
 + [ ] tools/upload: add ~+upload/open-remote-file~ command to open current file
       on the remote (with TRAMP)
 + [ ] tools/regex: PCRE regex editor, maybe ~re-builder~ & ~pcre2el~?
++ [X] core-popups: properly persist ~popup~ window parameter between sessions
 + [X] Fix crash in ~doom-update-package~ caused by unreachable dependencies
 + [X] core-popups: remove ~:noclone~: simply don't try to clone unreal buffers
 + [X] completion/ivy: flexible column width for ~+ivy/tasks~
diff --git a/core/core-popups.el b/core/core-popups.el
index 0784e608c..c9a09f41a 100644
--- a/core/core-popups.el
+++ b/core/core-popups.el
@@ -115,8 +115,7 @@ for :align t on every rule."
 ;; Tell `window-state-get' and `current-window-configuration' to recognize these
 ;; custom parameters. Helpful for `persp-mode' and persisting window configs
 ;; that have popups in them.
-(push (cons 'no-other-window 'writable) window-persistent-parameters)
-(dolist (param doom-popup-window-parameters)
+(dolist (param (cons 'popup doom-popup-window-parameters))
   (push (cons param 'writable) window-persistent-parameters))
 
 (defvar doom-popup-mode-map