X-Git-Url: http://git.scottworley.com/vopamoi/blobdiff_plain/a59fbe41a148be79ea3623cd9a07cecf11b67d6c..3a731557d09232dbf6871a33cff2cbeda42db880:/vopamoi.ts diff --git a/vopamoi.ts b/vopamoi.ts index fdc4723..01e761c 100644 --- a/vopamoi.ts +++ b/vopamoi.ts @@ -49,12 +49,11 @@ const Model = { const input = target.children[0] as HTMLInputElement; if ( input.value === target.getAttribute("data-description") && - input.selectionStart === 0 && + input.selectionStart === input.value.length && input.selectionEnd === input.value.length ) { // No local changes have actually been made yet. Change the contents of the edit box! input.value = newDescription; - input.select(); } else { // No great options. // Prefer not to interrupt the local user's edit. @@ -178,6 +177,11 @@ function UI() { } const ui = UI(); +enum CommitOrAbort { + Commit, + Abort, +} + function BrowserUI() { var currentViewState = "todo"; var taskFocusedBeforeJumpingToInput: HTMLElement | null = null; @@ -210,11 +214,10 @@ function BrowserUI() { task.textContent = ""; task.appendChild(input); input.focus(); - input.select(); event.preventDefault(); }, - completeEdit: function (event: Event) { + completeEdit: function (event: Event, resolution: CommitOrAbort = CommitOrAbort.Commit) { const input = event.target as HTMLInputElement; const task = input.parentElement!; const oldDescription = task.getAttribute("data-description")!; @@ -223,7 +226,7 @@ function BrowserUI() { task.removeChild(task.children[0]); task.removeAttribute("data-description"); task.focus(); - if (newDescription === oldDescription) { + if (newDescription === oldDescription || resolution === CommitOrAbort.Abort) { task.textContent = oldDescription; } else { ui.edit(task.getAttribute("data-created")!, newDescription, oldDescription); @@ -350,6 +353,7 @@ function handleKey(event: any) { if (event.key == "Escape") return browserUI.returnFocusAfterInput(); } else { if (event.key == "Enter") return browserUI.completeEdit(event); + if (event.key == "Escape") return browserUI.completeEdit(event, CommitOrAbort.Abort); } } else { if (inputState === InputState.Command) {