tweak(helm): show mode-line by default

Let's not hide the minibuffer. It may be detrimental in cases where
certain helm commands actually display helpful information there. And
rather than design edge cases for a package I don't dogfood, and push
minimalism onto helm users, I'll lean onto the defaults more instead.

Ref: #6676
Co-authored-by: johanwiden <johanwiden@users.noreply.github.com>

modules/completion/helm/config.el

@@ -51,7 +51,6 @@ Can be negative.")
   (setq helm-candidate-number-limit 50
         ;; Remove extraineous helm UI elements
         helm-display-header-line nil
-        helm-mode-line-string nil
         helm-ff-auto-update-initial-value nil
         helm-find-files-doc-header nil
         ;; Default helm window sizes
@@ -100,15 +99,6 @@ Can be negative.")
   :config
   (set-popup-rule! "^\\*helm" :vslot -100 :size 0.22 :ttl nil)
 
-  ;; Hide the modeline in helm windows as it serves little purpose.
-  (defun +helm--hide-mode-line (&rest _)
-    (with-current-buffer (helm-buffer-get)
-      (unless helm-mode-line-string
-        (hide-mode-line-mode +1))))
-  (add-hook 'helm-after-initialize-hook #'+helm--hide-mode-line)
-  (advice-add #'helm-display-mode-line :override #'+helm--hide-mode-line)
-  (advice-add #'helm-ag-show-status-default-mode-line :override #'ignore)
-
   ;; Hide minibuffer if `helm-echo-input-in-header-line'
   (add-hook 'helm-minibuffer-set-up-hook #'helm-hide-minibuffer-maybe)