fix(ui): remove delete shortcuts from recent lists
This commit is contained in:
@@ -124,17 +124,7 @@ const FolderSelectionView: Component<FolderSelectionViewProps> = (props) => {
|
|||||||
|
|
||||||
const normalizedKey = e.key.toLowerCase()
|
const normalizedKey = e.key.toLowerCase()
|
||||||
const isBrowseShortcut = (e.metaKey || e.ctrlKey) && !e.shiftKey && normalizedKey === "n"
|
const isBrowseShortcut = (e.metaKey || e.ctrlKey) && !e.shiftKey && normalizedKey === "n"
|
||||||
const blockedKeys = [
|
const blockedKeys = ["ArrowDown", "ArrowUp", "PageDown", "PageUp", "Home", "End", "Enter"]
|
||||||
"ArrowDown",
|
|
||||||
"ArrowUp",
|
|
||||||
"PageDown",
|
|
||||||
"PageUp",
|
|
||||||
"Home",
|
|
||||||
"End",
|
|
||||||
"Enter",
|
|
||||||
"Backspace",
|
|
||||||
"Delete",
|
|
||||||
]
|
|
||||||
|
|
||||||
if (isLoading()) {
|
if (isLoading()) {
|
||||||
if (isBrowseShortcut || blockedKeys.includes(e.key)) {
|
if (isBrowseShortcut || blockedKeys.includes(e.key)) {
|
||||||
@@ -192,21 +182,6 @@ const FolderSelectionView: Component<FolderSelectionViewProps> = (props) => {
|
|||||||
} else if (e.key === "Enter") {
|
} else if (e.key === "Enter") {
|
||||||
e.preventDefault()
|
e.preventDefault()
|
||||||
handleEnterKey()
|
handleEnterKey()
|
||||||
} else if (e.key === "Backspace" || e.key === "Delete") {
|
|
||||||
e.preventDefault()
|
|
||||||
if (listLength > 0 && focusMode() === "recent") {
|
|
||||||
if (activeTab() === "local") {
|
|
||||||
const folder = folders()[selectedIndex()]
|
|
||||||
if (folder) {
|
|
||||||
handleRemove(folder.path)
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
const server = serverList()[selectedIndex()]
|
|
||||||
if (server) {
|
|
||||||
removeRemoteServerProfile(server.id)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -953,10 +928,6 @@ const FolderSelectionView: Component<FolderSelectionViewProps> = (props) => {
|
|||||||
<kbd class="kbd">Enter</kbd>
|
<kbd class="kbd">Enter</kbd>
|
||||||
<span>{t("folderSelection.hints.select")}</span>
|
<span>{t("folderSelection.hints.select")}</span>
|
||||||
</div>
|
</div>
|
||||||
<div class="flex items-center gap-1.5">
|
|
||||||
<kbd class="kbd">Del</kbd>
|
|
||||||
<span>{t("folderSelection.hints.remove")}</span>
|
|
||||||
</div>
|
|
||||||
</Show>
|
</Show>
|
||||||
<div class="flex items-center gap-1.5">
|
<div class="flex items-center gap-1.5">
|
||||||
<Kbd shortcut="cmd+n" class="kbd-hint" />
|
<Kbd shortcut="cmd+n" class="kbd-hint" />
|
||||||
|
|||||||
@@ -171,9 +171,6 @@ const InstanceWelcomeView: Component<InstanceWelcomeViewProps> = (props) => {
|
|||||||
} else if (e.key === "Enter") {
|
} else if (e.key === "Enter") {
|
||||||
e.preventDefault()
|
e.preventDefault()
|
||||||
void handleEnterKey()
|
void handleEnterKey()
|
||||||
} else if (e.key === "Delete" || e.key === "Backspace") {
|
|
||||||
e.preventDefault()
|
|
||||||
void handleDeleteKey()
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@@ -187,29 +184,6 @@ const InstanceWelcomeView: Component<InstanceWelcomeViewProps> = (props) => {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
async function handleDeleteKey() {
|
|
||||||
const sessions = parentSessions()
|
|
||||||
const index = selectedIndex()
|
|
||||||
|
|
||||||
if (index >= sessions.length) {
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
await handleSessionDelete(sessions[index].id)
|
|
||||||
|
|
||||||
const updatedSessions = parentSessions()
|
|
||||||
if (updatedSessions.length === 0) {
|
|
||||||
setFocusMode("new-session")
|
|
||||||
setSelectedIndex(0)
|
|
||||||
return
|
|
||||||
}
|
|
||||||
|
|
||||||
const nextIndex = Math.min(index, updatedSessions.length - 1)
|
|
||||||
setSelectedIndex(nextIndex)
|
|
||||||
setFocusMode("sessions")
|
|
||||||
scrollToIndex(nextIndex)
|
|
||||||
}
|
|
||||||
|
|
||||||
onMount(() => {
|
onMount(() => {
|
||||||
window.addEventListener("keydown", handleKeyDown)
|
window.addEventListener("keydown", handleKeyDown)
|
||||||
|
|
||||||
@@ -562,10 +536,6 @@ const InstanceWelcomeView: Component<InstanceWelcomeViewProps> = (props) => {
|
|||||||
<kbd class="kbd">Enter</kbd>
|
<kbd class="kbd">Enter</kbd>
|
||||||
<span>{t("instanceWelcome.hints.resume")}</span>
|
<span>{t("instanceWelcome.hints.resume")}</span>
|
||||||
</div>
|
</div>
|
||||||
<div class="flex items-center gap-1.5">
|
|
||||||
<kbd class="kbd">Del</kbd>
|
|
||||||
<span>{t("instanceWelcome.hints.delete")}</span>
|
|
||||||
</div>
|
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user