Pop chatgpt shell into its own frame when not exists (with code and demo) about chatgpt-shell HOT 6 CLOSED

Nice idea! Have you looked at setting chatgpt-shell-display-function? By default, it uses pop-to-buffer-same-window but could be something else?

from chatgpt-shell.

Nice idea! Have you looked at setting chatgpt-shell-display-function? By default, it uses pop-to-buffer-same-window but could be something else?

I did code up a function to switch the shell in another frame, by default in the right half of your monitor:

(defun find-or-make-frame (fname)
    (condition-case
        nil
        (select-frame-by-name fname)
      (error (make-frame `((name . ,fname))))))
(defun display-chatgpt-shell-frame (bname)
    (let ((cur-f (selected-frame))
          (f (find-or-make-frame "chatgpt")))
      (select-frame-by-name "chatgpt")
      (pop-to-buffer-same-window bname)
      (set-frame-position f (/ (display-pixel-width) 2) 0)
      (set-frame-height f (frame-height cur-f))
      (set-frame-width f  (frame-width cur-f) 1)
      ))
   (setq shell-maker-display-function 'display-chatgpt-shell-frame)

from chatgpt-shell.

Ah sorry! I totally missed you using shell-maker-display-function. I blame reading from mobile ;-)

I'm thinking we can add the proposed function to the documentation as an example. I originally added the variable as someone had a different preference. The configuration option should be enough to cater for everyone's choice.

from chatgpt-shell.

But it would be nice if users can simply set a variable, not pasting a whole function and bloat their config files.

from chatgpt-shell.

Hear ya. The sweet spot isn't always clear. The challenge is that we are a diverse bunch of emacsers, so if we add all proposed functions to the package, the bloat will take place in the package itself. The compromise here is to add customization points and emacsers can choose to customize based on their preference. Hope that makes sense.

from chatgpt-shell.

Added sample to README.

ps. Hey, you wrote that amazing helm guide! I'm on ivy these days, but was on helm for quite some time thanks to that guide.

from chatgpt-shell.