fix: add cross-platform path delimiter support for file renaming and navigation logic #9

Merged
jamie merged 2 commits from v1.1.4 into main 2026-06-10 23:45:02 +01:00
Showing only changes of commit 75a4e6278c - Show all commits
+25 -6
View File
@@ -573,9 +573,10 @@ window.handleContextMenu = async (action) => {
const item = selectedItems[0]; const item = selectedItems[0];
const newName = prompt(`Enter new name for ${item.name}:`, item.name); const newName = prompt(`Enter new name for ${item.name}:`, item.name);
if (newName && newName !== item.name) { if (newName && newName !== item.name) {
const pathParts = item.path.split('/'); const sep = item.path.includes('\\') ? '\\' : '/';
const pathParts = item.path.split(/[/\\]/);
pathParts[pathParts.length - 1] = newName; pathParts[pathParts.length - 1] = newName;
const newPath = pathParts.join('/'); const newPath = pathParts.join(sep);
await Rename(currentConn, item.path, newPath); await Rename(currentConn, item.path, newPath);
refreshCurrentDir(); refreshCurrentDir();
@@ -595,9 +596,18 @@ window.handleContextMenu = async (action) => {
window.navigateUp = () => { window.navigateUp = () => {
if (!currentPath || currentPath === '/') return; if (!currentPath || currentPath === '/') return;
const parts = currentPath.split('/').filter(p => p); const startsWithSlash = currentPath.startsWith('/');
const sep = currentPath.includes('\\') ? '\\' : '/';
const parts = currentPath.split(/[/\\]/).filter(p => p);
parts.pop(); parts.pop();
currentPath = parts.length ? '/' + parts.join('/') : ''; if (parts.length === 0) {
currentPath = '';
} else {
currentPath = (startsWithSlash ? '/' : '') + parts.join(sep);
if (!startsWithSlash && parts.length === 1 && parts[0].endsWith(':')) {
currentPath += sep;
}
}
loadDirectory(currentConn, currentPath); loadDirectory(currentConn, currentPath);
}; };
@@ -641,9 +651,18 @@ window.loadTransferDestDirectory = async (connId, path) => {
tr.innerHTML = `<td>📁 ..</td>`; tr.innerHTML = `<td>📁 ..</td>`;
tr.style.cursor = 'pointer'; tr.style.cursor = 'pointer';
tr.ondblclick = () => { tr.ondblclick = () => {
const parts = path.split('/').filter(p => p); const startsWithSlash = path.startsWith('/');
const sep = path.includes('\\') ? '\\' : '/';
const parts = path.split(/[/\\]/).filter(p => p);
parts.pop(); parts.pop();
loadTransferDestDirectory(connId, parts.length ? '/' + parts.join('/') : ''); let newPath = '';
if (parts.length > 0) {
newPath = (startsWithSlash ? '/' : '') + parts.join(sep);
if (!startsWithSlash && parts.length === 1 && parts[0].endsWith(':')) {
newPath += sep;
}
}
loadTransferDestDirectory(connId, newPath);
}; };
tbody.appendChild(tr); tbody.appendChild(tr);
} }