Merge pull request #32 from hymkor/remove-c-esc

`edit`: Removed `c` and `Esc`

Author: hymkor

README.md

@@ -95,11 +95,12 @@ Supported commands
 - `EDIT [tablename [WHERE conditions...]]`
     - Start an [editor][csvi] to modify the selected records of the table.
     - In the editor, these keys are bound.
-        - `x` or `d`: set NULL to the current cell
-        - `ESC` + `y`: Apply changes and exit
-        - `ESC` + `n`: Discard changes and exit
-        - `q`: Now equivalent to `ESC`
-        - `c`: Apply changes and exit (deprecated)
+ 
+      | Key   | Bindings
+      |-------|-----------------------------
+      |`x`,`d`| Set NULL to the current cell
+      |`q`    | Quit editor with confirmation to apply or discard changes
+
     - Because the EDIT statement automatically generates SQL from data changed in the editor, it may not be able to properly represent SQL data for special types specific to individual databases. If you find it, we would appreciate it if you could [contact us](https://github.com/hymkor/sqlbless/issues/new).
 - `HOST command-line`
     - Executes an operating system command.

README_ja.md

@@ -93,11 +93,12 @@ SQL-Bless は、そうした現場での教訓から生まれた、安全で再
 - `EDIT [tablename [WHERE conditions...]]`
     - 選択したテーブルのレコードを修正するため [エディタ][csvi] を起動します
     - エディタ中では以下のキーが拡張されます
-        - `x` or `d`: セルに NULL をセットする
-        - `ESC`+`y`: 変更を適用して終了
-        - `ESC`+`n`: 修正を破棄して終了
-        - `q`: `ESC` と等価
-        - `c`: 変更を適用して終了(廃止予定)
+
+      | Key   | Bindings
+      |-------|-----------------------------
+      |`x`,`d`| セルに NULL をセットする
+      |`q`    | 変更を適用するかを確認してから終了
+
     - EDIT文は、エディターでの変更データから自動で SQL を生成する都合、個々のデータベース固有の特殊な型向けの SQL データをうまく表現できない場合があります。見つかりましたら、[ご連絡](https://github.com/hymkor/sqlbless/issues/new)いただけるとたすかります。
 - `HOST command-line`
     - OS コマンドを実行します

release_note_en.md

@@ -7,6 +7,7 @@ Release notes (English)
   - go-multiline-ny v0.22.3 →v0.22.4
   - go-readline-ny v1.13.0 → v1.14.1
   - go-ttyadapter v0.2.0 → v0.3.0
+- `edit`: Removed `c` and `Esc`. Use `q` to exit the editor, whether applying changes or not. (#32)
 
 v0.27.2
 -------

release_note_ja.md

@@ -7,6 +7,7 @@ Release notes (Japanese)
   - go-multiline-ny v0.22.3 →v0.22.4
   - go-readline-ny v1.13.0 → v1.14.1
   - go-ttyadapter v0.2.0 → v0.3.0
+- `edit`: `c` と `Esc` を廃止。変更の適用を伴う終了も`q` を使用してください。 (#32)
 
 v0.27.2
 -------

spread/view.go

@@ -92,29 +92,18 @@ func (viewer *Viewer) edit(title string, validate func(*csvi.CellValidatedEvent)
 		}
 	}
 
-	apply := func(app *csvi.KeyEventArgs) (*csvi.CommandResult, error) {
-		ch, err := app.MessageAndGetKey("Apply changes and quit ? [y/n] ")
-		if err == nil && (ch == "y" || ch == "Y") {
-			io.WriteString(app, "y\n")
-			applyChange = true
-			return &csvi.CommandResult{Quit: true}, nil
-		}
-		return &csvi.CommandResult{}, nil
-	}
 	keymap := map[string]func(*csvi.KeyEventArgs) (*csvi.CommandResult, error){
-		"\x1B": quit,
-		"q":    quit,
-		"c":    apply,
-		"x":    setNull,
-		"d":    setNull,
+		"q": quit,
+		"x": setNull,
+		"d": setNull,
 	}
 	for _, p := range viewer.OnEvents {
 		keymap[p.Key] = p.Handler
 	}
 	cfg := &csvi.Config{
 		Titles: []string{
 			toOneLine(title, titlePrefix, titleSuffix),
-			"ESC+\"y\": Apply changes & quit, ESC+\"n\": Discard changes & quit",
+			"Press `q` to quit: `y` to apply changes, `n` to discard.",
 		},
 		KeyMap:          keymap,
 		OnCellValidated: validate,