Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add information on split direction values #250

Closed

Conversation

themkat
Copy link

@themkat themkat commented Jan 2, 2025

I experienced some minor confusion while making Ghosty splits behave more like Emacs. Example values I ended up with:

keybind = ctrl+x>2=new_split:right
keybind = ctrl+x>3=new_split:down
keybind = super+left=goto_split:left
keybind = super+right=goto_split:right
keybind = super+up=goto_split:top
keybind = super+down=goto_split:bottom

The confusion arose from new_split using up/down while goto_split uses top/bottom. Felt like the docs could benefit from having some example values to avoid this confusion 🙂

Installed via homebrew on an m1 mac today, so should be the latest version. Feel free to reject this PR if not relevant anymore (or if I'm stupid) 🙂

EDIT: Sorry for being stupid. Seems like these are auto-generated from source. The live website is probably not updated after the doc was written.

Copy link

vercel bot commented Jan 2, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
website ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 2, 2025 6:26pm

Copy link

github-actions bot commented Jan 2, 2025

Note: This PR contains modifications to auto-generated sections of the documentation. These are generated from comments in ghostty's source code. Please submit changes to the upstream repository at https://github.com/ghostty-org/ghostty by modifying the appropriate source files

@themkat themkat closed this Jan 2, 2025
@themkat themkat deleted the docs/split_direction_valid_values branch January 2, 2025 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant