Skip to content

core/gtk: allow editing Ghostty config in a Ghostty window#11905

Open
jcollie wants to merge 4 commits intoghostty-org:mainfrom
jcollie:edit-config-new-window
Open

core/gtk: allow editing Ghostty config in a Ghostty window#11905
jcollie wants to merge 4 commits intoghostty-org:mainfrom
jcollie:edit-config-new-window

Conversation

@jcollie
Copy link
Copy Markdown
Member

@jcollie jcollie commented Mar 27, 2026

This PR extends the open_config keybind action to allow editing the Ghostty config in a new Ghostty window using the editor configured in $EDITOR or $VISUAL.

@jcollie jcollie requested review from a team as code owners March 27, 2026 22:33
@ghostty-bot ghostty-bot bot added the gtk Issues exclusive to GTK which are not OS-specific label Mar 27, 2026
@jcollie jcollie force-pushed the edit-config-new-window branch from e8abc94 to 8479d1e Compare March 27, 2026 23:12
jcollie added 3 commits March 28, 2026 19:26
This PR extends the `open_config` keybind action to allow editing the
Ghostty config in a new Ghostty window using the editor configured in
`$EDITOR` or `$VISUAL`.
@jcollie jcollie force-pushed the edit-config-new-window branch from 37f1b3c to db7d050 Compare March 29, 2026 00:32
@tristan957
Copy link
Copy Markdown
Member

I am not sure I am convinced of this PR. ghostty +edit-config opens directly in EDITOR and Open Configuration opens in the system's default editor. That seems pretty good to me. If we really wanted to allow this to be configurable, maybe we could just add a config key:

config-editor = cli:environment,ui:system

@jcollie
Copy link
Copy Markdown
Member Author

jcollie commented Mar 30, 2026

The main thing that this PR brings over running ghostty +edit-config is that the functionality of opening the config in $EDITOR is available from a keybind. Once PRs for +new-tab and (eventually) +new-split are merged I wanted to expand the options to include those choices.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gtk Issues exclusive to GTK which are not OS-specific

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants