From 2faa0c9bee71edf4b6eb6de65affb44be372fbde Mon Sep 17 00:00:00 2001
From: cryptocode <cryptocode@zolo.io>
Date: Fri, 10 May 2024 11:17:11 +0200
Subject: [PATCH] Rename Split Horizontally/Vertically to Split Right/Down

I make the following suggestion through a PR since I already tried this locally and have a branch ready:

- Rename Split Horizontally to Split Right
- Rename Split Vertically to Split Down

Rationale:

The terms horizontal and vertical are ambiguous in the context of splitting (does it mean the splitter is vertical, or are the panes organized vertically?)
Different apps pick different interpretations, so people get confused.

IntelliJ made the same change as above based on user feedback: https://youtrack.jetbrains.com/issue/IJPL-130991/Split-Vertically-and-Split-Horizontally-are-ambiguous-and-should-be-renamed

Ghostty is using the left/right nomenclature in other menu items. In the codebase, SplitDirection is already left/right.
---
 macos/Sources/App/macOS/MainMenu.xib                     | 8 ++++----
 macos/Sources/Features/Terminal/TerminalController.swift | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/macos/Sources/App/macOS/MainMenu.xib b/macos/Sources/App/macOS/MainMenu.xib
index cd627f762..1857fe75d 100644
--- a/macos/Sources/App/macOS/MainMenu.xib
+++ b/macos/Sources/App/macOS/MainMenu.xib
@@ -126,16 +126,16 @@
                                 </connections>
                             </menuItem>
                             <menuItem isSeparatorItem="YES" id="m54-Is-iLE"/>
-                            <menuItem title="Split Horizontally" id="VUR-Ld-nLx">
+                            <menuItem title="Split Right" id="VUR-Ld-nLx">
                                 <modifierMask key="keyEquivalentModifierMask"/>
                                 <connections>
-                                    <action selector="splitHorizontally:" target="-1" id="cv2-Xg-FR4"/>
+                                    <action selector="splitRight:" target="-1" id="cv2-Xg-FR4"/>
                                 </connections>
                             </menuItem>
-                            <menuItem title="Split Vertically" id="UDZ-4y-6xL">
+                            <menuItem title="Split Down" id="UDZ-4y-6xL">
                                 <modifierMask key="keyEquivalentModifierMask"/>
                                 <connections>
-                                    <action selector="splitVertically:" target="-1" id="c6x-CF-u52"/>
+                                    <action selector="splitDown:" target="-1" id="c6x-CF-u52"/>
                                 </connections>
                             </menuItem>
                             <menuItem isSeparatorItem="YES" id="sjq-M1-UGS"/>
diff --git a/macos/Sources/Features/Terminal/TerminalController.swift b/macos/Sources/Features/Terminal/TerminalController.swift
index cea140995..84b8ad01c 100644
--- a/macos/Sources/Features/Terminal/TerminalController.swift
+++ b/macos/Sources/Features/Terminal/TerminalController.swift
@@ -461,12 +461,12 @@ class TerminalController: NSWindowController, NSWindowDelegate,
         })
     }
     
-    @IBAction func splitHorizontally(_ sender: Any) {
+    @IBAction func splitRight(_ sender: Any) {
         guard let surface = focusedSurface?.surface else { return }
         ghostty.split(surface: surface, direction: GHOSTTY_SPLIT_RIGHT)
     }
     
-    @IBAction func splitVertically(_ sender: Any) {
+    @IBAction func splitDown(_ sender: Any) {
         guard let surface = focusedSurface?.surface else { return }
         ghostty.split(surface: surface, direction: GHOSTTY_SPLIT_DOWN)
     }