Full document, spreadsheet, slideshow, and diagram tooling
Merge pull request 'fix+feat: wire imports, command palette, row/col ops, context menu, virtual scroll' (#55) from fix/wire-modules-and-ux into main
+1654
-37
+281
src/command-palette.ts
+281
src/command-palette.ts
···
1
+
/**
2
+
* Command Palette (Cmd+K / Ctrl+K)
3
+
*
4
+
* A self-contained, fast command palette that works on all pages
5
+
* (landing, docs, sheets). Supports fuzzy search, keyboard navigation,
6
+
* categorized results, and light/dark themes.
7
+
*/
8
+
9
+
export interface PaletteAction {
10
+
id: string;
11
+
label: string;
12
+
category: 'document' | 'action';
13
+
icon?: string;
14
+
shortcut?: string;
15
+
action: () => void;
16
+
}
17
+
18
+
export interface PaletteConfig {
19
+
actions: PaletteAction[];
20
+
fetchDocuments?: () => Promise<PaletteAction[]>;
21
+
}
22
+
23
+
export interface CommandPaletteHandle {
24
+
open: () => void;
25
+
close: () => void;
26
+
destroy: () => void;
27
+
}
28
+
29
+
/**
30
+
* Fuzzy match: split query into words, each word must appear
31
+
* somewhere in the label (case insensitive). Order doesn't matter.
32
+
*/
33
+
export function fuzzyMatch(label: string, query: string): boolean {
34
+
if (!query || !query.trim()) return true;
35
+
const lower = label.toLowerCase();
36
+
const words = query.toLowerCase().trim().split(/\s+/);
37
+
return words.every(w => lower.includes(w));
38
+
}
39
+
40
+
/**
41
+
* Filter actions by query using fuzzy matching.
42
+
*/
43
+
export function filterActions(actions: PaletteAction[], query: string): PaletteAction[] {
44
+
if (!query || !query.trim()) return actions;
45
+
return actions.filter(a => fuzzyMatch(a.label, query));
46
+
}
47
+
48
+
/**
49
+
* Group actions by category, returning groups in order:
50
+
* 'action' first, then 'document'.
51
+
*/
52
+
export function groupByCategory(actions: PaletteAction[]): { category: string; items: PaletteAction[] }[] {
53
+
const actionItems = actions.filter(a => a.category === 'action');
54
+
const docItems = actions.filter(a => a.category === 'document');
55
+
const groups: { category: string; items: PaletteAction[] }[] = [];
56
+
if (actionItems.length > 0) groups.push({ category: 'Actions', items: actionItems });
57
+
if (docItems.length > 0) groups.push({ category: 'Documents', items: docItems });
58
+
return groups;
59
+
}
60
+
61
+
/**
62
+
* Compute the next selected index when navigating with arrow keys.
63
+
* Wraps around.
64
+
*/
65
+
export function navigateIndex(current: number, total: number, direction: 'up' | 'down'): number {
66
+
if (total === 0) return -1;
67
+
if (direction === 'down') {
68
+
return (current + 1) % total;
69
+
}
70
+
return (current - 1 + total) % total;
71
+
}
72
+
73
+
/**
74
+
* Create a command palette instance. Injects DOM, handles keyboard,
75
+
* renders results. Call `.destroy()` to clean up.
76
+
*/
77
+
export function createCommandPalette(config: PaletteConfig): CommandPaletteHandle {
78
+
// --- Build DOM ---
79
+
const backdrop = document.createElement('div');
80
+
backdrop.className = 'cmd-palette-backdrop';
81
+
backdrop.setAttribute('role', 'dialog');
82
+
backdrop.setAttribute('aria-modal', 'true');
83
+
backdrop.setAttribute('aria-label', 'Command palette');
84
+
85
+
const container = document.createElement('div');
86
+
container.className = 'cmd-palette';
87
+
88
+
const input = document.createElement('input');
89
+
input.className = 'cmd-palette-input';
90
+
input.type = 'text';
91
+
input.placeholder = 'Type a command or search...';
92
+
input.setAttribute('aria-label', 'Command palette search');
93
+
input.spellcheck = false;
94
+
input.autocomplete = 'off';
95
+
96
+
const resultsList = document.createElement('div');
97
+
resultsList.className = 'cmd-palette-results';
98
+
resultsList.setAttribute('role', 'listbox');
99
+
100
+
container.appendChild(input);
101
+
container.appendChild(resultsList);
102
+
backdrop.appendChild(container);
103
+
104
+
// --- State ---
105
+
let isOpen = false;
106
+
let allActions: PaletteAction[] = [...config.actions];
107
+
let filteredActions: PaletteAction[] = [];
108
+
let selectedIndex = 0;
109
+
let documentsFetched = false;
110
+
111
+
function render() {
112
+
const query = input.value;
113
+
filteredActions = filterActions(allActions, query);
114
+
const groups = groupByCategory(filteredActions);
115
+
116
+
if (filteredActions.length === 0) {
117
+
resultsList.innerHTML = '<div class="cmd-palette-empty">No results found</div>';
118
+
selectedIndex = -1;
119
+
return;
120
+
}
121
+
122
+
// Clamp selected index
123
+
if (selectedIndex >= filteredActions.length) selectedIndex = 0;
124
+
if (selectedIndex < 0) selectedIndex = 0;
125
+
126
+
let html = '';
127
+
let flatIndex = 0;
128
+
for (const group of groups) {
129
+
html += `<div class="cmd-palette-category">${escapeHtml(group.category)}</div>`;
130
+
for (const item of group.items) {
131
+
const isSelected = flatIndex === selectedIndex;
132
+
html += `<div class="cmd-palette-item${isSelected ? ' cmd-palette-item-selected' : ''}" data-index="${flatIndex}" role="option" aria-selected="${isSelected}">`;
133
+
if (item.icon) {
134
+
html += `<span class="cmd-palette-item-icon">${escapeHtml(item.icon)}</span>`;
135
+
}
136
+
html += `<span class="cmd-palette-item-label">${escapeHtml(item.label)}</span>`;
137
+
if (item.shortcut) {
138
+
html += `<span class="cmd-palette-item-shortcut">${escapeHtml(item.shortcut)}</span>`;
139
+
}
140
+
html += '</div>';
141
+
flatIndex++;
142
+
}
143
+
}
144
+
resultsList.innerHTML = html;
145
+
146
+
// Scroll selected into view
147
+
const selectedEl = resultsList.querySelector('.cmd-palette-item-selected');
148
+
if (selectedEl) {
149
+
selectedEl.scrollIntoView({ block: 'nearest' });
150
+
}
151
+
}
152
+
153
+
function executeSelected() {
154
+
if (selectedIndex >= 0 && selectedIndex < filteredActions.length) {
155
+
const action = filteredActions[selectedIndex];
156
+
close();
157
+
action.action();
158
+
}
159
+
}
160
+
161
+
function close() {
162
+
if (!isOpen) return;
163
+
isOpen = false;
164
+
backdrop.classList.remove('cmd-palette-open');
165
+
// Remove after animation
166
+
setTimeout(() => {
167
+
if (!isOpen) backdrop.remove();
168
+
}, 150);
169
+
}
170
+
171
+
function open() {
172
+
if (isOpen) return;
173
+
isOpen = true;
174
+
input.value = '';
175
+
selectedIndex = 0;
176
+
document.body.appendChild(backdrop);
177
+
// Force reflow for animation
178
+
backdrop.offsetHeight;
179
+
backdrop.classList.add('cmd-palette-open');
180
+
input.focus();
181
+
182
+
// Fetch documents on first open (or always if provided)
183
+
if (config.fetchDocuments && !documentsFetched) {
184
+
documentsFetched = true;
185
+
config.fetchDocuments().then(docs => {
186
+
// Merge, avoiding duplicate IDs
187
+
const existingIds = new Set(allActions.map(a => a.id));
188
+
for (const doc of docs) {
189
+
if (!existingIds.has(doc.id)) {
190
+
allActions.push(doc);
191
+
}
192
+
}
193
+
if (isOpen) render();
194
+
}).catch(() => {
195
+
// silently fail — actions still work
196
+
});
197
+
}
198
+
199
+
render();
200
+
}
201
+
202
+
// --- Event handlers ---
203
+
function onInput() {
204
+
selectedIndex = 0;
205
+
render();
206
+
}
207
+
208
+
function onKeydown(e: KeyboardEvent) {
209
+
if (e.key === 'Escape') {
210
+
e.preventDefault();
211
+
e.stopPropagation();
212
+
close();
213
+
} else if (e.key === 'ArrowDown') {
214
+
e.preventDefault();
215
+
selectedIndex = navigateIndex(selectedIndex, filteredActions.length, 'down');
216
+
render();
217
+
} else if (e.key === 'ArrowUp') {
218
+
e.preventDefault();
219
+
selectedIndex = navigateIndex(selectedIndex, filteredActions.length, 'up');
220
+
render();
221
+
} else if (e.key === 'Enter') {
222
+
e.preventDefault();
223
+
executeSelected();
224
+
}
225
+
}
226
+
227
+
function onBackdropClick(e: MouseEvent) {
228
+
if (e.target === backdrop) {
229
+
close();
230
+
}
231
+
}
232
+
233
+
function onResultClick(e: MouseEvent) {
234
+
const item = (e.target as HTMLElement).closest('.cmd-palette-item') as HTMLElement | null;
235
+
if (item) {
236
+
const index = parseInt(item.dataset.index || '0', 10);
237
+
selectedIndex = index;
238
+
executeSelected();
239
+
}
240
+
}
241
+
242
+
function onGlobalKeydown(e: KeyboardEvent) {
243
+
const mod = e.metaKey || e.ctrlKey;
244
+
if (mod && e.key.toLowerCase() === 'k') {
245
+
// Don't hijack Cmd+K when user is in a contenteditable or specific input
246
+
// that uses Cmd+K for link insertion. Only intercept if not already open.
247
+
e.preventDefault();
248
+
if (isOpen) {
249
+
close();
250
+
} else {
251
+
open();
252
+
}
253
+
}
254
+
}
255
+
256
+
// Attach listeners
257
+
input.addEventListener('input', onInput);
258
+
input.addEventListener('keydown', onKeydown);
259
+
backdrop.addEventListener('click', onBackdropClick);
260
+
resultsList.addEventListener('click', onResultClick);
261
+
document.addEventListener('keydown', onGlobalKeydown);
262
+
263
+
return {
264
+
open,
265
+
close,
266
+
destroy() {
267
+
close();
268
+
document.removeEventListener('keydown', onGlobalKeydown);
269
+
input.removeEventListener('input', onInput);
270
+
input.removeEventListener('keydown', onKeydown);
271
+
backdrop.removeEventListener('click', onBackdropClick);
272
+
resultsList.removeEventListener('click', onResultClick);
273
+
},
274
+
};
275
+
}
276
+
277
+
function escapeHtml(text: string): string {
278
+
const div = document.createElement('div');
279
+
div.textContent = text;
280
+
return div.innerHTML;
281
+
}
+134
src/css/app.css
+134
src/css/app.css
···
4879
4879
background: oklch(0.60 0.1 195 / 0.05);
4880
4880
}
4881
4881
}
4882
+
4883
+
/* ==========================================================
4884
+
Command Palette (Cmd+K / Ctrl+K)
4885
+
========================================================== */
4886
+
4887
+
.cmd-palette-backdrop {
4888
+
position: fixed;
4889
+
inset: 0;
4890
+
background: var(--color-modal-backdrop);
4891
+
display: flex;
4892
+
align-items: flex-start;
4893
+
justify-content: center;
4894
+
padding-top: min(20vh, 160px);
4895
+
z-index: 9999;
4896
+
opacity: 0;
4897
+
transition: opacity var(--transition-fast);
4898
+
}
4899
+
4900
+
.cmd-palette-backdrop.cmd-palette-open {
4901
+
opacity: 1;
4902
+
}
4903
+
4904
+
.cmd-palette {
4905
+
background: var(--color-surface);
4906
+
border: 1px solid var(--color-border);
4907
+
border-radius: var(--radius-lg);
4908
+
box-shadow: var(--shadow-lg);
4909
+
width: 90%;
4910
+
max-width: 600px;
4911
+
max-height: 60vh;
4912
+
display: flex;
4913
+
flex-direction: column;
4914
+
overflow: hidden;
4915
+
transform: translateY(-8px) scale(0.98);
4916
+
transition: transform var(--transition-fast), opacity var(--transition-fast);
4917
+
opacity: 0;
4918
+
}
4919
+
4920
+
.cmd-palette-backdrop.cmd-palette-open .cmd-palette {
4921
+
transform: translateY(0) scale(1);
4922
+
opacity: 1;
4923
+
}
4924
+
4925
+
.cmd-palette-input {
4926
+
width: 100%;
4927
+
padding: var(--space-md) var(--space-lg);
4928
+
border: none;
4929
+
border-bottom: 1px solid var(--color-border);
4930
+
background: transparent;
4931
+
color: var(--color-text);
4932
+
font: inherit;
4933
+
font-size: 1.05rem;
4934
+
outline: none;
4935
+
}
4936
+
4937
+
.cmd-palette-input::placeholder {
4938
+
color: var(--color-text-faint);
4939
+
}
4940
+
4941
+
.cmd-palette-results {
4942
+
overflow-y: auto;
4943
+
flex: 1;
4944
+
padding: var(--space-xs) 0;
4945
+
}
4946
+
4947
+
.cmd-palette-category {
4948
+
padding: var(--space-sm) var(--space-lg) var(--space-xs);
4949
+
font-family: var(--font-body);
4950
+
font-size: 0.7rem;
4951
+
font-weight: 600;
4952
+
text-transform: uppercase;
4953
+
letter-spacing: 0.05em;
4954
+
color: var(--color-text-faint);
4955
+
}
4956
+
4957
+
.cmd-palette-item {
4958
+
display: flex;
4959
+
align-items: center;
4960
+
gap: var(--space-sm);
4961
+
padding: var(--space-sm) var(--space-lg);
4962
+
cursor: pointer;
4963
+
color: var(--color-text);
4964
+
font-size: 0.9rem;
4965
+
transition: background var(--transition-fast);
4966
+
}
4967
+
4968
+
.cmd-palette-item:hover,
4969
+
.cmd-palette-item-selected {
4970
+
background: var(--color-hover);
4971
+
}
4972
+
4973
+
.cmd-palette-item-selected {
4974
+
background: var(--color-hover);
4975
+
}
4976
+
4977
+
.cmd-palette-item-icon {
4978
+
flex-shrink: 0;
4979
+
width: 1.5rem;
4980
+
text-align: center;
4981
+
font-size: 0.95rem;
4982
+
color: var(--color-text-muted);
4983
+
}
4984
+
4985
+
.cmd-palette-item-label {
4986
+
flex: 1;
4987
+
min-width: 0;
4988
+
overflow: hidden;
4989
+
text-overflow: ellipsis;
4990
+
white-space: nowrap;
4991
+
}
4992
+
4993
+
.cmd-palette-item-shortcut {
4994
+
flex-shrink: 0;
4995
+
font-size: 0.75rem;
4996
+
font-family: var(--font-mono);
4997
+
color: var(--color-text-faint);
4998
+
background: var(--color-surface-alt);
4999
+
padding: 2px 6px;
5000
+
border-radius: var(--radius-sm);
5001
+
border: 1px solid var(--color-border);
5002
+
}
5003
+
5004
+
.cmd-palette-empty {
5005
+
padding: var(--space-xl) var(--space-lg);
5006
+
text-align: center;
5007
+
color: var(--color-text-faint);
5008
+
font-size: 0.9rem;
5009
+
}
5010
+
5011
+
@media print {
5012
+
.cmd-palette-backdrop {
5013
+
display: none !important;
5014
+
}
5015
+
}
+49
src/docs/main.ts
+49
src/docs/main.ts
···
56
56
import { SLASH_COMMAND_ITEMS, SlashMenuState, filterCommands, PLACEHOLDER_EMPTY, PLACEHOLDER_BLOCK } from './slash-menu.js';
57
57
import { createSlashCommands, getCommandExecutor } from './extensions/slash-commands.js';
58
58
import { BlockHandleState, BLOCK_HANDLE_ACTIONS, TURN_INTO_ITEMS, filterTurnIntoItems, BLOCK_HANDLE_ICON, BLOCK_HANDLE_ADD_ICON } from './block-handle.js';
59
+
import { createCommandPalette, type PaletteAction } from '../command-palette.js';
59
60
60
61
// --- Resolve document ID and encryption key ---
61
62
const pathParts = location.pathname.split('/').filter(Boolean);
···
2110
2111
{ keys: ['\u2318', '\u21e7', 'F'], label: 'Zen mode' },
2111
2112
],
2112
2113
});
2114
+
2115
+
// --- Command Palette ---
2116
+
createCommandPalette({
2117
+
actions: [
2118
+
{ id: 'back', label: 'Back to Documents', category: 'action', icon: '\u2190', action: () => { window.location.href = '/'; } },
2119
+
{ id: 'new-doc', label: 'New Document', category: 'action', icon: '\u270e', action: () => { window.open('/', '_blank'); } },
2120
+
{ id: 'new-sheet', label: 'New Spreadsheet', category: 'action', icon: '\u25a6', action: () => { window.open('/', '_blank'); } },
2121
+
{ id: 'bold', label: 'Bold', category: 'action', icon: 'B', shortcut: '\u2318B', action: () => editor.chain().focus().toggleBold().run() },
2122
+
{ id: 'italic', label: 'Italic', category: 'action', icon: 'I', shortcut: '\u2318I', action: () => editor.chain().focus().toggleItalic().run() },
2123
+
{ id: 'underline', label: 'Underline', category: 'action', icon: 'U', shortcut: '\u2318U', action: () => editor.chain().focus().toggleUnderline().run() },
2124
+
{ id: 'table', label: 'Insert Table', category: 'action', icon: '\u2637', action: () => editor.chain().focus().insertTable({ rows: 3, cols: 3, withHeaderRow: true }).run() },
2125
+
{ id: 'markdown', label: 'Toggle Markdown', category: 'action', icon: 'MD', shortcut: '\u2318\u21e7M', action: () => mdToggle.toggle() },
2126
+
{ id: 'zen', label: 'Zen Mode', category: 'action', icon: '\u2022', shortcut: '\u2318\u21e7F', action: () => toggleZenMode() },
2127
+
{ id: 'export-pdf', label: 'Export PDF', category: 'action', icon: '\u2193', shortcut: '\u2318\u21e7P', action: () => doExportPdf() },
2128
+
{ id: 'import', label: 'Import File', category: 'action', icon: '\u2191', action: () => importFile() },
2129
+
{ id: 'find', label: 'Find & Replace', category: 'action', icon: '\u2315', shortcut: '\u2318F', action: () => { editor.commands.openSearch(); updateFindBar(); } },
2130
+
],
2131
+
fetchDocuments: async (): Promise<PaletteAction[]> => {
2132
+
try {
2133
+
const res = await fetch('/api/documents');
2134
+
const docs = await res.json();
2135
+
const keys = JSON.parse(localStorage.getItem('tools-keys') || '{}');
2136
+
const results: PaletteAction[] = [];
2137
+
for (const doc of docs) {
2138
+
if (doc.id === docId) continue; // skip current doc
2139
+
const keyStr = keys[doc.id];
2140
+
if (!keyStr) continue;
2141
+
const path = doc.type === 'doc' ? '/docs' : '/sheets';
2142
+
let name = 'Encrypted Document';
2143
+
if (keyStr && doc.name_encrypted) {
2144
+
try {
2145
+
const k = await importKey(keyStr);
2146
+
const encBytes = Uint8Array.from(atob(doc.name_encrypted), (c: string) => c.charCodeAt(0));
2147
+
name = await decryptString(encBytes, k);
2148
+
} catch { /* can't decrypt */ }
2149
+
}
2150
+
results.push({
2151
+
id: `doc-${doc.id}`,
2152
+
label: name,
2153
+
category: 'document',
2154
+
icon: doc.type === 'doc' ? '\u270e' : '\u25a6',
2155
+
action: () => { window.location.href = `${path}/${doc.id}#${keyStr}`; },
2156
+
});
2157
+
}
2158
+
return results;
2159
+
} catch { return []; }
2160
+
},
2161
+
});
+36
src/landing.ts
+36
src/landing.ts
···
1
1
import type { DocumentMeta, TrashEntry, Folder, FolderAssignments, StarMap, SortLabels } from './landing-types.js';
2
2
import { generateKey, exportKey, importKey, decryptString } from './lib/crypto.js';
3
+
import { createCommandPalette, type PaletteAction } from './command-palette.js';
3
4
import {
4
5
sortDocuments,
5
6
toggleStar,
···
742
743
} catch (err) {
743
744
showToast('Failed to create document for import', 4000, true);
744
745
}
746
+
});
747
+
748
+
// --- Command Palette ---
749
+
createCommandPalette({
750
+
actions: [
751
+
{
752
+
id: 'new-doc',
753
+
label: 'New Document',
754
+
category: 'action',
755
+
icon: '\u270e',
756
+
action: () => createDocument('doc'),
757
+
},
758
+
{
759
+
id: 'new-sheet',
760
+
label: 'New Spreadsheet',
761
+
category: 'action',
762
+
icon: '\u25a6',
763
+
action: () => createDocument('sheet'),
764
+
},
765
+
],
766
+
fetchDocuments: async (): Promise<PaletteAction[]> => {
767
+
const keys = JSON.parse(localStorage.getItem('tools-keys') || '{}');
768
+
return allDocs
769
+
.filter(doc => doc._keyStr)
770
+
.map(doc => {
771
+
const path = doc.type === 'doc' ? '/docs' : '/sheets';
772
+
return {
773
+
id: `doc-${doc.id}`,
774
+
label: doc._decryptedName || 'Encrypted Document',
775
+
category: 'document' as const,
776
+
icon: doc.type === 'doc' ? '\u270e' : '\u25a6',
777
+
action: () => { window.location.href = `${path}/${doc.id}#${doc._keyStr}`; },
778
+
};
779
+
});
780
+
},
745
781
});
746
782
747
783
// --- Init ---
+169
-37
src/sheets/main.ts
+169
-37
src/sheets/main.ts
···
25
25
import { extractFormulaRanges, assignRangeColors, renderGridHighlights, clearGridHighlights } from './range-highlight.js';
26
26
import { detectCurrentFunction, renderTooltip, hideTooltip } from './formula-tooltip.js';
27
27
import { extractFormat, applyFormat } from './format-painter.js';
28
+
import { calculateVisibleRange, calculateSpacerHeight, DEFAULT_ROW_HEIGHT, DEFAULT_BUFFER_ROWS } from './virtual-scroll.js';
29
+
import { insertRow as rowColInsertRow, deleteRow as rowColDeleteRow, insertColumn as rowColInsertColumn, deleteColumn as rowColDeleteColumn } from './row-col-ops.js';
30
+
import { createContextMenu, buildSheetsCellItems, buildSheetsColumnHeaderItems, buildSheetsRowHeaderItems, SEPARATOR } from '../lib/context-menu.js';
31
+
import type { MenuItem, MenuItemConfig } from '../lib/context-menu.js';
28
32
29
33
// --- Constants ---
30
34
const DEFAULT_ROWS = 100;
···
295
299
const cfRules = getCfRulesArray();
296
300
const stripedEnabled = getStripedRows();
297
301
298
-
// --- Body rows ---
299
-
for (let r = 1; r <= rowCount; r++) {
302
+
// --- Virtual scrolling: calculate visible row range (#146) ---
303
+
const viewportHeight = sheetContainer ? sheetContainer.clientHeight : 600;
304
+
const scrollTop = sheetContainer ? sheetContainer.scrollTop : 0;
305
+
const visibleRange = calculateVisibleRange({
306
+
scrollTop,
307
+
viewportHeight,
308
+
totalRows: rowCount,
309
+
rowHeight: bodyRowHeight,
310
+
bufferRows: 5,
311
+
});
312
+
313
+
// Always render frozen rows + visible range
314
+
const renderStartRow = Math.max(freezeR + 1, visibleRange.startRow);
315
+
const renderEndRow = visibleRange.endRow;
316
+
317
+
// Top spacer: account for skipped rows above the visible range (after frozen rows)
318
+
const skippedAbove = renderStartRow - freezeR - 1;
319
+
if (skippedAbove > 0) {
320
+
html += '<tr class="virtual-spacer-top" style="height:' + (skippedAbove * bodyRowHeight) + 'px"><td colspan="' + (colCount + 1) + '"></td></tr>';
321
+
}
322
+
323
+
// --- Body rows (frozen rows always rendered, then visible range) ---
324
+
const rowsToRender = [];
325
+
for (let r = 1; r <= freezeR; r++) rowsToRender.push(r);
326
+
for (let r = renderStartRow; r <= renderEndRow; r++) rowsToRender.push(r);
327
+
328
+
for (const r of rowsToRender) {
300
329
html += '<tr>';
301
330
const rhCls = ['row-header'];
302
331
if (r <= freezeR) {
···
367
396
}
368
397
html += '</tr>';
369
398
}
399
+
400
+
// Bottom spacer: account for skipped rows below the visible range
401
+
const skippedBelow = rowCount - renderEndRow;
402
+
if (skippedBelow > 0) {
403
+
html += '<tr class="virtual-spacer-bottom" style="height:' + (skippedBelow * bodyRowHeight) + 'px"><td colspan="' + (colCount + 1) + '"></td></tr>';
404
+
}
405
+
370
406
html += '</tbody>';
371
407
grid.innerHTML = html;
372
408
···
1371
1407
}
1372
1408
closeAllDropdowns();
1373
1409
});
1410
+
1411
+
// --- Row/Column Insert/Delete operations (#113) ---
1412
+
function doInsertRow(rowIndex) {
1413
+
const sheet = getActiveSheet();
1414
+
const colCount = sheet.get('colCount') || DEFAULT_COLS;
1415
+
ydoc.transact(() => {
1416
+
rowColInsertRow(getCells, setCellData, rowIndex, colCount);
1417
+
});
1418
+
sheet.set('rowCount', (sheet.get('rowCount') || DEFAULT_ROWS) + 1);
1419
+
evalCache.clear(); invalidateRecalcEngine();
1420
+
renderGrid();
1421
+
}
1422
+
1423
+
function doDeleteRow(rowIndex) {
1424
+
const sheet = getActiveSheet();
1425
+
const rowCount = sheet.get('rowCount') || DEFAULT_ROWS;
1426
+
if (rowCount <= 1) return;
1427
+
const colCount = sheet.get('colCount') || DEFAULT_COLS;
1428
+
ydoc.transact(() => {
1429
+
rowColDeleteRow(getCells, setCellData, rowIndex, colCount);
1430
+
});
1431
+
sheet.set('rowCount', rowCount - 1);
1432
+
evalCache.clear(); invalidateRecalcEngine();
1433
+
renderGrid();
1434
+
}
1435
+
1436
+
function doInsertColumn(colIndex) {
1437
+
const sheet = getActiveSheet();
1438
+
const rowCount = sheet.get('rowCount') || DEFAULT_ROWS;
1439
+
ydoc.transact(() => {
1440
+
rowColInsertColumn(getCells, setCellData, colIndex, rowCount);
1441
+
});
1442
+
sheet.set('colCount', (sheet.get('colCount') || DEFAULT_COLS) + 1);
1443
+
evalCache.clear(); invalidateRecalcEngine();
1444
+
renderGrid();
1445
+
}
1446
+
1447
+
function doDeleteColumn(colIndex) {
1448
+
const sheet = getActiveSheet();
1449
+
const colCount = sheet.get('colCount') || DEFAULT_COLS;
1450
+
if (colCount <= 1) return;
1451
+
const rowCount = sheet.get('rowCount') || DEFAULT_ROWS;
1452
+
ydoc.transact(() => {
1453
+
rowColDeleteColumn(getCells, setCellData, colIndex, rowCount);
1454
+
});
1455
+
sheet.set('colCount', colCount - 1);
1456
+
evalCache.clear(); invalidateRecalcEngine();
1457
+
renderGrid();
1458
+
}
1374
1459
1375
1460
// --- Frozen panes toolbar (Issue #7) ---
1376
1461
document.getElementById('tb-freeze-rows').addEventListener('click', () => { const rows = selectedCell.row - 1; setFreezeRows(rows > 0 ? rows : 0); renderGrid(); closeAllDropdowns(); });
···
3059
3144
if (td) hideNoteTooltip();
3060
3145
});
3061
3146
3062
-
// Right-click context menu for notes
3147
+
// Right-click context menu (#149 — wired-up actions, #113 — row/col insert/delete)
3148
+
let _activeContextMenu = null;
3149
+
3150
+
function hideActiveContextMenu() {
3151
+
if (_activeContextMenu) {
3152
+
_activeContextMenu.destroy();
3153
+
_activeContextMenu = null;
3154
+
}
3155
+
}
3156
+
3063
3157
grid.addEventListener('contextmenu', (e) => {
3064
-
const td = e.target.closest('td[data-id]');
3065
-
if (!td) return;
3066
3158
e.preventDefault();
3159
+
hideActiveContextMenu();
3067
3160
3068
-
const cid = td.dataset.id;
3069
-
const notes = getNotesObject();
3070
-
const noteExists = hasNote(notes, cid);
3161
+
const colHeader = e.target.closest('thead th[data-col]');
3162
+
const rowHeader = e.target.closest('th.row-header[data-row]');
3163
+
const td = e.target.closest('td[data-id]');
3071
3164
3072
-
// Create context menu
3073
-
const existing = document.querySelector('.cell-context-menu');
3074
-
if (existing) existing.remove();
3165
+
let items;
3166
+
if (colHeader) {
3167
+
// Column header right-click
3168
+
const col = parseInt(colHeader.dataset.col);
3169
+
items = [
3170
+
{ label: 'Sort A \u2192 Z', icon: '\u2191', action: () => { selectedCell = { col, row: 1 }; const sheet = getActiveSheet(); selectionRange = { startCol: col, startRow: 1, endCol: col, endRow: sheet.get('rowCount') || DEFAULT_ROWS }; sortColumn(true); } },
3171
+
{ label: 'Sort Z \u2192 A', icon: '\u2193', action: () => { selectedCell = { col, row: 1 }; const sheet = getActiveSheet(); selectionRange = { startCol: col, startRow: 1, endCol: col, endRow: sheet.get('rowCount') || DEFAULT_ROWS }; sortColumn(false); } },
3172
+
SEPARATOR,
3173
+
{ label: 'Insert Column Left', action: () => doInsertColumn(col) },
3174
+
{ label: 'Insert Column Right', action: () => doInsertColumn(col + 1) },
3175
+
{ label: 'Delete Column', action: () => doDeleteColumn(col) },
3176
+
];
3177
+
} else if (rowHeader) {
3178
+
// Row header right-click
3179
+
const row = parseInt(rowHeader.dataset.row);
3180
+
items = [
3181
+
{ label: 'Insert Row Above', action: () => doInsertRow(row) },
3182
+
{ label: 'Insert Row Below', action: () => doInsertRow(row + 1) },
3183
+
{ label: 'Delete Row', action: () => doDeleteRow(row) },
3184
+
];
3185
+
} else if (td) {
3186
+
// Cell right-click
3187
+
const col = parseInt(td.dataset.col);
3188
+
const row = parseInt(td.dataset.row);
3189
+
const cid = td.dataset.id;
3190
+
const notes = getNotesObject();
3191
+
const noteExists = hasNote(notes, cid);
3192
+
items = [
3193
+
{ label: 'Cut', icon: '\u2702', shortcut: '\u2318X', action: () => { copySelection(); deleteSelectedCells(); } },
3194
+
{ label: 'Copy', icon: '\u29C9', shortcut: '\u2318C', action: () => copySelection() },
3195
+
{ label: 'Paste', icon: '\uD83D\uDCCB', shortcut: '\u2318V', action: () => { navigator.clipboard.readText().then(text => pasteAtSelection(text)).catch(() => {}); } },
3196
+
SEPARATOR,
3197
+
{ label: 'Insert Row Above', action: () => doInsertRow(row) },
3198
+
{ label: 'Insert Row Below', action: () => doInsertRow(row + 1) },
3199
+
{ label: 'Insert Column Left', action: () => doInsertColumn(col) },
3200
+
{ label: 'Insert Column Right', action: () => doInsertColumn(col + 1) },
3201
+
SEPARATOR,
3202
+
{ label: 'Delete Row', action: () => doDeleteRow(row) },
3203
+
{ label: 'Delete Column', action: () => doDeleteColumn(col) },
3204
+
SEPARATOR,
3205
+
{ label: 'Clear Cells', action: () => deleteSelectedCells() },
3206
+
{ label: noteExists ? 'Edit Note' : 'Add Note', icon: '\uD83D\uDCDD', action: () => showNoteDialog(cid) },
3207
+
...(noteExists ? [{ label: 'Delete Note', action: () => { setNoteInYjs(cid, null); renderNoteIndicators(); } }] : []),
3208
+
];
3209
+
} else {
3210
+
return;
3211
+
}
3075
3212
3076
-
const menu = document.createElement('div');
3077
-
menu.className = 'cell-context-menu';
3078
-
menu.style.cssText = 'position:fixed;z-index:1002;background:var(--color-bg);border:1px solid var(--color-border-strong);border-radius:var(--radius-md);box-shadow:var(--shadow-md);padding:2px 0;min-width:140px;';
3079
-
menu.innerHTML = '<button class="toolbar-dropdown-item" data-action="add-note">'
3080
-
+ '<span class="item-label">' + (noteExists ? 'Edit note' : 'Add note') + '</span></button>'
3081
-
+ (noteExists ? '<button class="toolbar-dropdown-item" data-action="delete-note"><span class="item-label">Delete note</span></button>' : '');
3082
-
3083
-
menu.style.left = e.clientX + 'px';
3084
-
menu.style.top = e.clientY + 'px';
3085
-
document.body.appendChild(menu);
3086
-
3087
-
menu.addEventListener('click', (ev) => {
3088
-
const action = ev.target.closest('[data-action]')?.dataset.action;
3089
-
if (action === 'add-note') {
3090
-
showNoteDialog(cid);
3091
-
} else if (action === 'delete-note') {
3092
-
setNoteInYjs(cid, null);
3093
-
renderNoteIndicators();
3094
-
}
3095
-
menu.remove();
3096
-
});
3213
+
const ctxMenu = createContextMenu(items);
3214
+
document.body.appendChild(ctxMenu.el);
3215
+
ctxMenu.show(e.clientX, e.clientY);
3216
+
_activeContextMenu = ctxMenu;
3097
3217
3098
3218
// Close on click outside
3099
-
const closeMenu = (ev) => {
3100
-
if (!menu.contains(ev.target)) {
3101
-
menu.remove();
3102
-
document.removeEventListener('click', closeMenu);
3219
+
const closeHandler = (ev) => {
3220
+
if (!ctxMenu.el.contains(ev.target)) {
3221
+
hideActiveContextMenu();
3222
+
document.removeEventListener('mousedown', closeHandler);
3103
3223
}
3104
3224
};
3105
-
setTimeout(() => document.addEventListener('click', closeMenu), 0);
3225
+
setTimeout(() => document.addEventListener('mousedown', closeHandler), 0);
3106
3226
});
3107
3227
3108
3228
// Observe notes changes from collaborators
3109
3229
getNotesMap().observe(() => renderNoteIndicators());
3230
+
3231
+
// --- Virtual scroll: re-render on scroll (#146) ---
3232
+
let _scrollRenderTimer = null;
3233
+
if (sheetContainer) {
3234
+
sheetContainer.addEventListener('scroll', () => {
3235
+
if (_scrollRenderTimer) return;
3236
+
_scrollRenderTimer = requestAnimationFrame(() => {
3237
+
_scrollRenderTimer = null;
3238
+
if (!editingCell) renderGrid();
3239
+
});
3240
+
});
3241
+
}
3110
3242
3111
3243
// --- Initial render ---
3112
3244
ensureSheet(0);
+334
src/sheets/row-col-ops.ts
+334
src/sheets/row-col-ops.ts
···
1
+
/**
2
+
* Row/Column Insert/Delete operations for Tools Sheets.
3
+
*
4
+
* Pure functions that handle:
5
+
* - Shifting cell data when rows/columns are inserted or deleted
6
+
* - Adjusting formula references (including ranges and absolute refs)
7
+
* - Operating on Yjs-backed cell maps
8
+
*/
9
+
10
+
import { parseRef, colToLetter, cellId } from './formulas.js';
11
+
12
+
// --- Formula reference adjustment ---
13
+
14
+
/**
15
+
* Adjust cell references in a formula string when rows or columns are
16
+
* inserted or deleted.
17
+
*
18
+
* - Handles single cell refs (A1, $A1, A$1, $A$1)
19
+
* - Handles range refs (A1:B10)
20
+
* - Leaves cross-sheet refs (Sheet1!A1) untouched
21
+
* - Absolute refs ($) are NOT shifted
22
+
* - References to deleted rows/cols become #REF!
23
+
*/
24
+
export function adjustFormulaRefs(
25
+
formula: string,
26
+
change: { type: 'row' | 'col'; index: number; delta: number },
27
+
): string {
28
+
// Match cell references with optional $ for absolute refs, including ranges
29
+
// Pattern: optional $, column letters, optional $, row digits
30
+
// Also match cross-sheet refs to skip them
31
+
const cellRefPattern = /(?:(?:'[^']*'|[A-Za-z_]\w*)!)?\$?[A-Z]{1,3}\$?\d+/g;
32
+
33
+
return formula.replace(cellRefPattern, (match) => {
34
+
// Skip cross-sheet references
35
+
if (match.includes('!')) return match;
36
+
37
+
return adjustSingleRef(match, change);
38
+
});
39
+
}
40
+
41
+
/**
42
+
* Adjust a single cell reference like A1, $A$1, $A1, A$1.
43
+
*/
44
+
function adjustSingleRef(
45
+
ref: string,
46
+
change: { type: 'row' | 'col'; index: number; delta: number },
47
+
): string {
48
+
const parsed = parseRefWithDollars(ref);
49
+
if (!parsed) return ref;
50
+
51
+
const { colAbs, colLetter, rowAbs, row } = parsed;
52
+
const col = letterToColNum(colLetter);
53
+
54
+
if (change.type === 'row') {
55
+
if (rowAbs) return ref; // $1 => absolute, don't shift
56
+
if (change.delta > 0) {
57
+
// Inserting rows: shift refs at or below index down
58
+
if (row >= change.index) {
59
+
return buildRef(colAbs, colLetter, false, row + change.delta);
60
+
}
61
+
} else {
62
+
// Deleting rows (delta is negative)
63
+
const deleteStart = change.index;
64
+
const deleteCount = Math.abs(change.delta);
65
+
const deleteEnd = deleteStart + deleteCount - 1;
66
+
if (row >= deleteStart && row <= deleteEnd) {
67
+
return '#REF!';
68
+
}
69
+
if (row > deleteEnd) {
70
+
return buildRef(colAbs, colLetter, false, row + change.delta);
71
+
}
72
+
}
73
+
} else {
74
+
// Column change
75
+
if (colAbs) return ref; // $A => absolute, don't shift
76
+
if (change.delta > 0) {
77
+
// Inserting columns: shift refs at or right of index
78
+
if (col >= change.index) {
79
+
return buildRef(false, colToLetter(col + change.delta), rowAbs, row);
80
+
}
81
+
} else {
82
+
// Deleting columns
83
+
const deleteStart = change.index;
84
+
const deleteCount = Math.abs(change.delta);
85
+
const deleteEnd = deleteStart + deleteCount - 1;
86
+
if (col >= deleteStart && col <= deleteEnd) {
87
+
return '#REF!';
88
+
}
89
+
if (col > deleteEnd) {
90
+
return buildRef(false, colToLetter(col + change.delta), rowAbs, row);
91
+
}
92
+
}
93
+
}
94
+
95
+
return ref;
96
+
}
97
+
98
+
interface ParsedRef {
99
+
colAbs: boolean;
100
+
colLetter: string;
101
+
rowAbs: boolean;
102
+
row: number;
103
+
}
104
+
105
+
function parseRefWithDollars(ref: string): ParsedRef | null {
106
+
const match = ref.match(/^(\$?)([A-Z]{1,3})(\$?)(\d+)$/);
107
+
if (!match) return null;
108
+
return {
109
+
colAbs: match[1] === '$',
110
+
colLetter: match[2],
111
+
rowAbs: match[3] === '$',
112
+
row: parseInt(match[4]),
113
+
};
114
+
}
115
+
116
+
function buildRef(colAbs: boolean, colLetter: string, rowAbs: boolean, row: number): string {
117
+
return (colAbs ? '$' : '') + colLetter + (rowAbs ? '$' : '') + row;
118
+
}
119
+
120
+
function letterToColNum(letter: string): number {
121
+
let col = 0;
122
+
for (let i = 0; i < letter.length; i++) {
123
+
col = col * 26 + (letter.charCodeAt(i) - 64);
124
+
}
125
+
return col;
126
+
}
127
+
128
+
// --- Cell shifting ---
129
+
130
+
/**
131
+
* Given a list of cell IDs and a row/col change, return a map of old ID => new ID
132
+
* for cells that need to be moved.
133
+
*/
134
+
export function getCellsToShift(
135
+
cellIds: string[],
136
+
change: { type: 'row' | 'col'; index: number; delta: number },
137
+
): Map<string, string> {
138
+
const result = new Map<string, string>();
139
+
140
+
for (const id of cellIds) {
141
+
const ref = parseRef(id);
142
+
if (!ref) continue;
143
+
144
+
if (change.type === 'row') {
145
+
if (change.delta > 0) {
146
+
// Insert: shift cells at or below index
147
+
if (ref.row >= change.index) {
148
+
result.set(id, cellId(ref.col, ref.row + change.delta));
149
+
}
150
+
} else {
151
+
// Delete: remove cells in deleted range, shift those below up
152
+
const deleteStart = change.index;
153
+
const deleteCount = Math.abs(change.delta);
154
+
const deleteEnd = deleteStart + deleteCount - 1;
155
+
if (ref.row >= deleteStart && ref.row <= deleteEnd) {
156
+
result.set(id, ''); // empty string means deleted
157
+
} else if (ref.row > deleteEnd) {
158
+
result.set(id, cellId(ref.col, ref.row + change.delta));
159
+
}
160
+
}
161
+
} else {
162
+
if (change.delta > 0) {
163
+
// Insert: shift cells at or right of index
164
+
if (ref.col >= change.index) {
165
+
result.set(id, cellId(ref.col + change.delta, ref.row));
166
+
}
167
+
} else {
168
+
// Delete: remove cells in deleted range, shift those right of it left
169
+
const deleteStart = change.index;
170
+
const deleteCount = Math.abs(change.delta);
171
+
const deleteEnd = deleteStart + deleteCount - 1;
172
+
if (ref.col >= deleteStart && ref.col <= deleteEnd) {
173
+
result.set(id, ''); // deleted
174
+
} else if (ref.col > deleteEnd) {
175
+
result.set(id, cellId(ref.col + change.delta, ref.row));
176
+
}
177
+
}
178
+
}
179
+
}
180
+
181
+
return result;
182
+
}
183
+
184
+
// --- Yjs data operations ---
185
+
186
+
// Minimal interface for the cell map (compatible with Y.Map)
187
+
interface CellMap {
188
+
get(key: string): unknown;
189
+
set(key: string, value: unknown): void;
190
+
has(key: string): boolean;
191
+
delete(key: string): void;
192
+
forEach(callback: (value: unknown, key: string) => void): void;
193
+
keys?(): IterableIterator<string>;
194
+
}
195
+
196
+
/**
197
+
* Collect all cell IDs from the cell map.
198
+
*/
199
+
function getAllCellIds(cells: CellMap): string[] {
200
+
const ids: string[] = [];
201
+
cells.forEach((_, key) => ids.push(key));
202
+
return ids;
203
+
}
204
+
205
+
/**
206
+
* Read cell data, adjust formulas, write to new positions.
207
+
* This is the core engine used by insert/delete operations.
208
+
*
209
+
* The `getCells` callback returns the Yjs cell map.
210
+
* The `setCellData` callback writes cell data (creating the Y.Map entry if needed).
211
+
*/
212
+
function shiftCells(
213
+
getCells: () => CellMap,
214
+
setCellData: (id: string, data: { v?: unknown; f?: string; s?: unknown }) => void,
215
+
change: { type: 'row' | 'col'; index: number; delta: number },
216
+
): void {
217
+
const cells = getCells();
218
+
const allIds = getAllCellIds(cells);
219
+
const shiftMap = getCellsToShift(allIds, change);
220
+
221
+
// Read all affected cell data first (before modifying)
222
+
const cellDataCache = new Map<string, { v: unknown; f: string; s: unknown }>();
223
+
for (const [oldId] of shiftMap) {
224
+
if (cells.has(oldId)) {
225
+
const yCell = cells.get(oldId) as { get(k: string): unknown } | null;
226
+
if (yCell && typeof yCell === 'object' && 'get' in yCell) {
227
+
cellDataCache.set(oldId, {
228
+
v: yCell.get('v') ?? '',
229
+
f: yCell.get('f') ?? '',
230
+
s: yCell.get('s') ?? '',
231
+
});
232
+
}
233
+
}
234
+
}
235
+
236
+
// Delete old positions
237
+
for (const [oldId, newId] of shiftMap) {
238
+
if (cells.has(oldId)) cells.delete(oldId);
239
+
// If newId is empty, cell was in deleted range - already deleted above
240
+
}
241
+
242
+
// Write to new positions with adjusted formulas
243
+
const writtenIds = new Set<string>();
244
+
for (const [oldId, newId] of shiftMap) {
245
+
if (!newId) continue; // deleted cell
246
+
const data = cellDataCache.get(oldId);
247
+
if (!data) continue;
248
+
249
+
let formula = data.f as string;
250
+
if (formula) {
251
+
formula = adjustFormulaRefs(formula, change);
252
+
}
253
+
254
+
setCellData(newId, {
255
+
v: data.v,
256
+
f: formula,
257
+
s: typeof data.s === 'string' && data.s ? JSON.parse(data.s as string) : data.s,
258
+
});
259
+
writtenIds.add(newId);
260
+
}
261
+
262
+
// Also adjust formulas in cells that did NOT move (and were not just written)
263
+
const updatedCells = getCells();
264
+
const remainingIds: string[] = [];
265
+
updatedCells.forEach((_, key) => {
266
+
if (!shiftMap.has(key) && !writtenIds.has(key)) {
267
+
remainingIds.push(key);
268
+
}
269
+
});
270
+
271
+
for (const id of remainingIds) {
272
+
const yCell = updatedCells.get(id) as { get(k: string): unknown; set(k: string, v: unknown): void } | null;
273
+
if (!yCell || typeof yCell !== 'object' || !('get' in yCell)) continue;
274
+
const formula = yCell.get('f') as string;
275
+
if (!formula) continue;
276
+
277
+
const adjusted = adjustFormulaRefs(formula, change);
278
+
if (adjusted !== formula) {
279
+
yCell.set('f', adjusted);
280
+
}
281
+
}
282
+
}
283
+
284
+
/**
285
+
* Insert a row at the given index (1-based). All cells at rowIndex and below
286
+
* shift down by 1. Formulas are updated.
287
+
*/
288
+
export function insertRow(
289
+
getCells: () => CellMap,
290
+
setCellData: (id: string, data: { v?: unknown; f?: string; s?: unknown }) => void,
291
+
rowIndex: number,
292
+
_colCount: number,
293
+
): void {
294
+
shiftCells(getCells, setCellData, { type: 'row', index: rowIndex, delta: 1 });
295
+
}
296
+
297
+
/**
298
+
* Delete a row at the given index (1-based). All cells below shift up by 1.
299
+
* Cells in the deleted row are removed. Formulas are updated.
300
+
*/
301
+
export function deleteRow(
302
+
getCells: () => CellMap,
303
+
setCellData: (id: string, data: { v?: unknown; f?: string; s?: unknown }) => void,
304
+
rowIndex: number,
305
+
_colCount: number,
306
+
): void {
307
+
shiftCells(getCells, setCellData, { type: 'row', index: rowIndex, delta: -1 });
308
+
}
309
+
310
+
/**
311
+
* Insert a column at the given index (1-based). All cells at colIndex and right
312
+
* shift right by 1. Formulas are updated.
313
+
*/
314
+
export function insertColumn(
315
+
getCells: () => CellMap,
316
+
setCellData: (id: string, data: { v?: unknown; f?: string; s?: unknown }) => void,
317
+
colIndex: number,
318
+
_rowCount: number,
319
+
): void {
320
+
shiftCells(getCells, setCellData, { type: 'col', index: colIndex, delta: 1 });
321
+
}
322
+
323
+
/**
324
+
* Delete a column at the given index (1-based). All cells right of it shift left by 1.
325
+
* Cells in the deleted column are removed. Formulas are updated.
326
+
*/
327
+
export function deleteColumn(
328
+
getCells: () => CellMap,
329
+
setCellData: (id: string, data: { v?: unknown; f?: string; s?: unknown }) => void,
330
+
colIndex: number,
331
+
_rowCount: number,
332
+
): void {
333
+
shiftCells(getCells, setCellData, { type: 'col', index: colIndex, delta: -1 });
334
+
}
+148
tests/command-palette.test.ts
+148
tests/command-palette.test.ts
···
1
+
import { describe, it, expect } from 'vitest';
2
+
import {
3
+
fuzzyMatch,
4
+
filterActions,
5
+
groupByCategory,
6
+
navigateIndex,
7
+
type PaletteAction,
8
+
} from '../src/command-palette.js';
9
+
10
+
// --- Test data ---
11
+
const actions: PaletteAction[] = [
12
+
{ id: 'new-doc', label: 'New Document', category: 'action', icon: '+', action: () => {} },
13
+
{ id: 'new-sheet', label: 'New Spreadsheet', category: 'action', icon: '+', action: () => {} },
14
+
{ id: 'bold', label: 'Bold', category: 'action', shortcut: '\u2318B', action: () => {} },
15
+
{ id: 'italic', label: 'Italic', category: 'action', shortcut: '\u2318I', action: () => {} },
16
+
{ id: 'doc-1', label: 'Q1 Budget Report', category: 'document', icon: '\u270e', action: () => {} },
17
+
{ id: 'doc-2', label: 'Meeting Notes March', category: 'document', icon: '\u270e', action: () => {} },
18
+
{ id: 'doc-3', label: 'Project Alpha Spec', category: 'document', icon: '\u25a6', action: () => {} },
19
+
];
20
+
21
+
// --- fuzzyMatch ---
22
+
describe('fuzzyMatch', () => {
23
+
it('matches everything for empty query', () => {
24
+
expect(fuzzyMatch('Hello World', '')).toBe(true);
25
+
expect(fuzzyMatch('Hello World', ' ')).toBe(true);
26
+
});
27
+
28
+
it('matches single word (case insensitive)', () => {
29
+
expect(fuzzyMatch('New Document', 'document')).toBe(true);
30
+
expect(fuzzyMatch('New Document', 'DOCUMENT')).toBe(true);
31
+
expect(fuzzyMatch('New Document', 'Document')).toBe(true);
32
+
});
33
+
34
+
it('matches multiple words (all must appear, any order)', () => {
35
+
expect(fuzzyMatch('Q1 Budget Report', 'budget report')).toBe(true);
36
+
expect(fuzzyMatch('Q1 Budget Report', 'report budget')).toBe(true);
37
+
expect(fuzzyMatch('Q1 Budget Report', 'q1 report')).toBe(true);
38
+
});
39
+
40
+
it('fails when a word is not found', () => {
41
+
expect(fuzzyMatch('Q1 Budget Report', 'budget xyz')).toBe(false);
42
+
expect(fuzzyMatch('New Document', 'spreadsheet')).toBe(false);
43
+
});
44
+
45
+
it('handles partial word matches', () => {
46
+
expect(fuzzyMatch('New Document', 'doc')).toBe(true);
47
+
expect(fuzzyMatch('New Document', 'ne do')).toBe(true);
48
+
});
49
+
50
+
it('handles whitespace in query', () => {
51
+
expect(fuzzyMatch('Bold', ' bold ')).toBe(true);
52
+
});
53
+
});
54
+
55
+
// --- filterActions ---
56
+
describe('filterActions', () => {
57
+
it('returns all actions for empty query', () => {
58
+
expect(filterActions(actions, '')).toEqual(actions);
59
+
expect(filterActions(actions, ' ')).toEqual(actions);
60
+
});
61
+
62
+
it('filters by label match', () => {
63
+
const result = filterActions(actions, 'bold');
64
+
expect(result).toHaveLength(1);
65
+
expect(result[0].id).toBe('bold');
66
+
});
67
+
68
+
it('filters multiple matches', () => {
69
+
const result = filterActions(actions, 'new');
70
+
expect(result).toHaveLength(2);
71
+
expect(result.map(a => a.id)).toEqual(['new-doc', 'new-sheet']);
72
+
});
73
+
74
+
it('returns empty for no matches', () => {
75
+
const result = filterActions(actions, 'zzzzz');
76
+
expect(result).toEqual([]);
77
+
});
78
+
79
+
it('matches across categories', () => {
80
+
const result = filterActions(actions, 'spec');
81
+
expect(result).toHaveLength(1);
82
+
expect(result[0].category).toBe('document');
83
+
});
84
+
});
85
+
86
+
// --- groupByCategory ---
87
+
describe('groupByCategory', () => {
88
+
it('groups actions and documents separately', () => {
89
+
const groups = groupByCategory(actions);
90
+
expect(groups).toHaveLength(2);
91
+
expect(groups[0].category).toBe('Actions');
92
+
expect(groups[1].category).toBe('Documents');
93
+
});
94
+
95
+
it('puts actions before documents', () => {
96
+
const groups = groupByCategory(actions);
97
+
expect(groups[0].category).toBe('Actions');
98
+
expect(groups[1].category).toBe('Documents');
99
+
});
100
+
101
+
it('omits empty categories', () => {
102
+
const onlyDocs = actions.filter(a => a.category === 'document');
103
+
const groups = groupByCategory(onlyDocs);
104
+
expect(groups).toHaveLength(1);
105
+
expect(groups[0].category).toBe('Documents');
106
+
});
107
+
108
+
it('returns empty array for no actions', () => {
109
+
expect(groupByCategory([])).toEqual([]);
110
+
});
111
+
112
+
it('preserves order within categories', () => {
113
+
const groups = groupByCategory(actions);
114
+
const actionIds = groups[0].items.map(i => i.id);
115
+
expect(actionIds).toEqual(['new-doc', 'new-sheet', 'bold', 'italic']);
116
+
});
117
+
});
118
+
119
+
// --- navigateIndex ---
120
+
describe('navigateIndex', () => {
121
+
it('moves down', () => {
122
+
expect(navigateIndex(0, 5, 'down')).toBe(1);
123
+
expect(navigateIndex(3, 5, 'down')).toBe(4);
124
+
});
125
+
126
+
it('wraps around going down', () => {
127
+
expect(navigateIndex(4, 5, 'down')).toBe(0);
128
+
});
129
+
130
+
it('moves up', () => {
131
+
expect(navigateIndex(3, 5, 'up')).toBe(2);
132
+
expect(navigateIndex(1, 5, 'up')).toBe(0);
133
+
});
134
+
135
+
it('wraps around going up', () => {
136
+
expect(navigateIndex(0, 5, 'up')).toBe(4);
137
+
});
138
+
139
+
it('returns -1 for empty list', () => {
140
+
expect(navigateIndex(0, 0, 'down')).toBe(-1);
141
+
expect(navigateIndex(0, 0, 'up')).toBe(-1);
142
+
});
143
+
144
+
it('handles single item', () => {
145
+
expect(navigateIndex(0, 1, 'down')).toBe(0);
146
+
expect(navigateIndex(0, 1, 'up')).toBe(0);
147
+
});
148
+
});
+503
tests/row-col-ops.test.ts
+503
tests/row-col-ops.test.ts
···
1
+
import { describe, it, expect } from 'vitest';
2
+
import {
3
+
adjustFormulaRefs,
4
+
getCellsToShift,
5
+
insertRow,
6
+
deleteRow,
7
+
insertColumn,
8
+
deleteColumn,
9
+
} from '../src/sheets/row-col-ops.js';
10
+
11
+
// ============================================================
12
+
// Helper: simple in-memory cell map for testing
13
+
// ============================================================
14
+
15
+
interface MockCellData {
16
+
v: unknown;
17
+
f: string;
18
+
s: unknown;
19
+
}
20
+
21
+
function createMockCellMap(initial: Record<string, MockCellData> = {}) {
22
+
const store = new Map<string, MockCell>();
23
+
24
+
class MockCell {
25
+
private data = new Map<string, unknown>();
26
+
constructor(cellData?: MockCellData) {
27
+
if (cellData) {
28
+
this.data.set('v', cellData.v);
29
+
this.data.set('f', cellData.f);
30
+
this.data.set('s', cellData.s ?? '');
31
+
}
32
+
}
33
+
get(key: string) { return this.data.get(key); }
34
+
set(key: string, value: unknown) { this.data.set(key, value); }
35
+
}
36
+
37
+
for (const [id, data] of Object.entries(initial)) {
38
+
store.set(id, new MockCell(data));
39
+
}
40
+
41
+
const cellMap = {
42
+
get(key: string) { return store.get(key); },
43
+
set(key: string, value: unknown) { store.set(key, value as MockCell); },
44
+
has(key: string) { return store.has(key); },
45
+
delete(key: string) { store.delete(key); },
46
+
forEach(cb: (value: unknown, key: string) => void) { store.forEach(cb); },
47
+
};
48
+
49
+
function getCells() { return cellMap; }
50
+
51
+
function setCellData(id: string, data: { v?: unknown; f?: string; s?: unknown }) {
52
+
let cell: MockCell;
53
+
if (store.has(id)) {
54
+
cell = store.get(id)!;
55
+
} else {
56
+
cell = new MockCell();
57
+
store.set(id, cell);
58
+
}
59
+
if (data.v !== undefined) cell.set('v', data.v);
60
+
if (data.f !== undefined) cell.set('f', data.f);
61
+
if (data.s !== undefined) cell.set('s', typeof data.s === 'object' ? JSON.stringify(data.s) : data.s);
62
+
}
63
+
64
+
function getCellData(id: string): MockCellData | null {
65
+
const cell = store.get(id);
66
+
if (!cell) return null;
67
+
return {
68
+
v: cell.get('v') ?? '',
69
+
f: (cell.get('f') as string) ?? '',
70
+
s: cell.get('s') ?? '',
71
+
};
72
+
}
73
+
74
+
return { getCells, setCellData, getCellData, store };
75
+
}
76
+
77
+
// ============================================================
78
+
// adjustFormulaRefs — Row insertion
79
+
// ============================================================
80
+
81
+
describe('adjustFormulaRefs — row insertion', () => {
82
+
it('shifts a cell ref below the insert point down', () => {
83
+
expect(adjustFormulaRefs('A5+A10', { type: 'row', index: 3, delta: 1 }))
84
+
.toBe('A6+A11');
85
+
});
86
+
87
+
it('shifts a cell ref at the insert point down', () => {
88
+
expect(adjustFormulaRefs('A3', { type: 'row', index: 3, delta: 1 }))
89
+
.toBe('A4');
90
+
});
91
+
92
+
it('does not shift a cell ref above the insert point', () => {
93
+
expect(adjustFormulaRefs('A1+A2', { type: 'row', index: 3, delta: 1 }))
94
+
.toBe('A1+A2');
95
+
});
96
+
97
+
it('shifts multi-column refs', () => {
98
+
expect(adjustFormulaRefs('B5+C5', { type: 'row', index: 3, delta: 1 }))
99
+
.toBe('B6+C6');
100
+
});
101
+
102
+
it('does not shift absolute row refs ($)', () => {
103
+
expect(adjustFormulaRefs('A$5', { type: 'row', index: 3, delta: 1 }))
104
+
.toBe('A$5');
105
+
});
106
+
107
+
it('shifts non-absolute column with absolute row unchanged', () => {
108
+
// $A5 means absolute column, relative row -> row shifts
109
+
expect(adjustFormulaRefs('$A5', { type: 'row', index: 3, delta: 1 }))
110
+
.toBe('$A6');
111
+
});
112
+
113
+
it('handles complex formula with mixed refs', () => {
114
+
expect(adjustFormulaRefs('SUM(A1:A10)+B3', { type: 'row', index: 5, delta: 1 }))
115
+
.toBe('SUM(A1:A11)+B3');
116
+
});
117
+
});
118
+
119
+
// ============================================================
120
+
// adjustFormulaRefs — Row deletion
121
+
// ============================================================
122
+
123
+
describe('adjustFormulaRefs — row deletion', () => {
124
+
it('shifts a cell ref below the deleted row up', () => {
125
+
expect(adjustFormulaRefs('A5', { type: 'row', index: 3, delta: -1 }))
126
+
.toBe('A4');
127
+
});
128
+
129
+
it('turns a ref to the deleted row into #REF!', () => {
130
+
expect(adjustFormulaRefs('A3', { type: 'row', index: 3, delta: -1 }))
131
+
.toBe('#REF!');
132
+
});
133
+
134
+
it('does not shift a cell ref above the deleted row', () => {
135
+
expect(adjustFormulaRefs('A1+A2', { type: 'row', index: 3, delta: -1 }))
136
+
.toBe('A1+A2');
137
+
});
138
+
139
+
it('does not shift absolute row ref ($)', () => {
140
+
expect(adjustFormulaRefs('A$3', { type: 'row', index: 3, delta: -1 }))
141
+
.toBe('A$3');
142
+
});
143
+
});
144
+
145
+
// ============================================================
146
+
// adjustFormulaRefs — Column insertion
147
+
// ============================================================
148
+
149
+
describe('adjustFormulaRefs — column insertion', () => {
150
+
it('shifts a cell ref at or right of the insert point right', () => {
151
+
expect(adjustFormulaRefs('C1+D1', { type: 'col', index: 3, delta: 1 }))
152
+
.toBe('D1+E1');
153
+
});
154
+
155
+
it('does not shift a cell ref left of the insert point', () => {
156
+
expect(adjustFormulaRefs('A1+B1', { type: 'col', index: 3, delta: 1 }))
157
+
.toBe('A1+B1');
158
+
});
159
+
160
+
it('does not shift absolute column ref ($)', () => {
161
+
expect(adjustFormulaRefs('$C1', { type: 'col', index: 3, delta: 1 }))
162
+
.toBe('$C1');
163
+
});
164
+
165
+
it('shifts relative column with absolute row', () => {
166
+
expect(adjustFormulaRefs('C$1', { type: 'col', index: 3, delta: 1 }))
167
+
.toBe('D$1');
168
+
});
169
+
});
170
+
171
+
// ============================================================
172
+
// adjustFormulaRefs — Column deletion
173
+
// ============================================================
174
+
175
+
describe('adjustFormulaRefs — column deletion', () => {
176
+
it('shifts a cell ref right of the deleted column left', () => {
177
+
expect(adjustFormulaRefs('D1', { type: 'col', index: 3, delta: -1 }))
178
+
.toBe('C1');
179
+
});
180
+
181
+
it('turns a ref to the deleted column into #REF!', () => {
182
+
expect(adjustFormulaRefs('C1', { type: 'col', index: 3, delta: -1 }))
183
+
.toBe('#REF!');
184
+
});
185
+
186
+
it('does not shift a cell ref left of the deleted column', () => {
187
+
expect(adjustFormulaRefs('A1+B1', { type: 'col', index: 3, delta: -1 }))
188
+
.toBe('A1+B1');
189
+
});
190
+
});
191
+
192
+
// ============================================================
193
+
// adjustFormulaRefs — Cross-sheet refs (should NOT be touched)
194
+
// ============================================================
195
+
196
+
describe('adjustFormulaRefs — cross-sheet refs', () => {
197
+
it('does not modify cross-sheet references', () => {
198
+
expect(adjustFormulaRefs("Sheet2!A5", { type: 'row', index: 3, delta: 1 }))
199
+
.toBe("Sheet2!A5");
200
+
});
201
+
202
+
it('does not modify quoted cross-sheet references', () => {
203
+
expect(adjustFormulaRefs("'My Sheet'!A5", { type: 'row', index: 3, delta: 1 }))
204
+
.toBe("'My Sheet'!A5");
205
+
});
206
+
});
207
+
208
+
// ============================================================
209
+
// adjustFormulaRefs — Range references
210
+
// ============================================================
211
+
212
+
describe('adjustFormulaRefs — range references', () => {
213
+
it('shifts both ends of a range when inserting row inside it', () => {
214
+
// A1:A10 with row 5 inserted -> A1 stays, A10 shifts to A11
215
+
expect(adjustFormulaRefs('SUM(A1:A10)', { type: 'row', index: 5, delta: 1 }))
216
+
.toBe('SUM(A1:A11)');
217
+
});
218
+
219
+
it('shifts entire range when insert is above it', () => {
220
+
expect(adjustFormulaRefs('SUM(A5:A10)', { type: 'row', index: 3, delta: 1 }))
221
+
.toBe('SUM(A6:A11)');
222
+
});
223
+
224
+
it('does not shift range when insert is below it', () => {
225
+
expect(adjustFormulaRefs('SUM(A1:A3)', { type: 'row', index: 5, delta: 1 }))
226
+
.toBe('SUM(A1:A3)');
227
+
});
228
+
229
+
it('adjusts column ranges on column insert', () => {
230
+
expect(adjustFormulaRefs('SUM(C1:E1)', { type: 'col', index: 3, delta: 1 }))
231
+
.toBe('SUM(D1:F1)');
232
+
});
233
+
});
234
+
235
+
// ============================================================
236
+
// adjustFormulaRefs — Fully absolute refs ($A$1)
237
+
// ============================================================
238
+
239
+
describe('adjustFormulaRefs — fully absolute refs', () => {
240
+
it('does not shift $A$1 on row insert', () => {
241
+
expect(adjustFormulaRefs('$A$1', { type: 'row', index: 1, delta: 1 }))
242
+
.toBe('$A$1');
243
+
});
244
+
245
+
it('does not shift $A$1 on column insert', () => {
246
+
expect(adjustFormulaRefs('$A$1', { type: 'col', index: 1, delta: 1 }))
247
+
.toBe('$A$1');
248
+
});
249
+
250
+
it('does not shift $C$5 on row delete', () => {
251
+
expect(adjustFormulaRefs('$C$5', { type: 'row', index: 5, delta: -1 }))
252
+
.toBe('$C$5');
253
+
});
254
+
255
+
it('does not shift $C$5 on column delete', () => {
256
+
expect(adjustFormulaRefs('$C$5', { type: 'col', index: 3, delta: -1 }))
257
+
.toBe('$C$5');
258
+
});
259
+
});
260
+
261
+
// ============================================================
262
+
// adjustFormulaRefs — Edge cases
263
+
// ============================================================
264
+
265
+
describe('adjustFormulaRefs — edge cases', () => {
266
+
it('handles empty formula', () => {
267
+
expect(adjustFormulaRefs('', { type: 'row', index: 1, delta: 1 }))
268
+
.toBe('');
269
+
});
270
+
271
+
it('handles formula with no cell refs', () => {
272
+
expect(adjustFormulaRefs('42+10', { type: 'row', index: 1, delta: 1 }))
273
+
.toBe('42+10');
274
+
});
275
+
276
+
it('handles string literals without modifying them', () => {
277
+
// String in formula like "A1" should not be treated as ref
278
+
// In practice formulas use quotes which don't match our pattern
279
+
expect(adjustFormulaRefs('"hello"', { type: 'row', index: 1, delta: 1 }))
280
+
.toBe('"hello"');
281
+
});
282
+
283
+
it('handles row 1 insert at row 1', () => {
284
+
expect(adjustFormulaRefs('A1', { type: 'row', index: 1, delta: 1 }))
285
+
.toBe('A2');
286
+
});
287
+
});
288
+
289
+
// ============================================================
290
+
// getCellsToShift
291
+
// ============================================================
292
+
293
+
describe('getCellsToShift', () => {
294
+
it('maps cells at and below insert row to new positions', () => {
295
+
const result = getCellsToShift(['A1', 'A2', 'A3'], { type: 'row', index: 2, delta: 1 });
296
+
expect(result.get('A2')).toBe('A3');
297
+
expect(result.get('A3')).toBe('A4');
298
+
expect(result.has('A1')).toBe(false); // above insert, no change
299
+
});
300
+
301
+
it('maps deleted row cells to empty string', () => {
302
+
const result = getCellsToShift(['A1', 'A2', 'A3'], { type: 'row', index: 2, delta: -1 });
303
+
expect(result.get('A2')).toBe(''); // deleted
304
+
expect(result.get('A3')).toBe('A2'); // shifted up
305
+
expect(result.has('A1')).toBe(false); // above, no change
306
+
});
307
+
308
+
it('maps cells at and right of insert col to new positions', () => {
309
+
const result = getCellsToShift(['A1', 'B1', 'C1'], { type: 'col', index: 2, delta: 1 });
310
+
expect(result.get('B1')).toBe('C1');
311
+
expect(result.get('C1')).toBe('D1');
312
+
expect(result.has('A1')).toBe(false);
313
+
});
314
+
315
+
it('maps deleted column cells to empty string', () => {
316
+
const result = getCellsToShift(['A1', 'B1', 'C1'], { type: 'col', index: 2, delta: -1 });
317
+
expect(result.get('B1')).toBe(''); // deleted
318
+
expect(result.get('C1')).toBe('B1'); // shifted left
319
+
expect(result.has('A1')).toBe(false);
320
+
});
321
+
});
322
+
323
+
// ============================================================
324
+
// insertRow — integration with mock cell map
325
+
// ============================================================
326
+
327
+
describe('insertRow', () => {
328
+
it('shifts cells down and adjusts formulas', () => {
329
+
const { getCells, setCellData, getCellData } = createMockCellMap({
330
+
'A1': { v: 10, f: '', s: '' },
331
+
'A2': { v: 20, f: '', s: '' },
332
+
'A3': { v: '', f: 'A1+A2', s: '' },
333
+
});
334
+
335
+
insertRow(getCells, setCellData, 2, 3);
336
+
337
+
// A1 stays (above insert)
338
+
expect(getCellData('A1')?.v).toBe(10);
339
+
// A2 moved to A3
340
+
expect(getCellData('A3')?.v).toBe(20);
341
+
// A3 (formula) moved to A4, formula adjusted
342
+
expect(getCellData('A4')?.f).toBe('A1+A3');
343
+
// A2 should be empty now (vacated)
344
+
expect(getCellData('A2')).toBeNull();
345
+
});
346
+
347
+
it('does not affect cells above the insert point', () => {
348
+
const { getCells, setCellData, getCellData } = createMockCellMap({
349
+
'A1': { v: 'stay', f: '', s: '' },
350
+
'A5': { v: 'move', f: '', s: '' },
351
+
});
352
+
353
+
insertRow(getCells, setCellData, 3, 1);
354
+
355
+
expect(getCellData('A1')?.v).toBe('stay');
356
+
expect(getCellData('A6')?.v).toBe('move');
357
+
});
358
+
});
359
+
360
+
// ============================================================
361
+
// deleteRow — integration with mock cell map
362
+
// ============================================================
363
+
364
+
describe('deleteRow', () => {
365
+
it('removes the target row and shifts cells up', () => {
366
+
const { getCells, setCellData, getCellData } = createMockCellMap({
367
+
'A1': { v: 10, f: '', s: '' },
368
+
'A2': { v: 'delete me', f: '', s: '' },
369
+
'A3': { v: 30, f: '', s: '' },
370
+
});
371
+
372
+
deleteRow(getCells, setCellData, 2, 3);
373
+
374
+
expect(getCellData('A1')?.v).toBe(10);
375
+
expect(getCellData('A2')?.v).toBe(30);
376
+
expect(getCellData('A3')).toBeNull();
377
+
});
378
+
379
+
it('adjusts formulas that referenced the deleted row to #REF!', () => {
380
+
const { getCells, setCellData, getCellData } = createMockCellMap({
381
+
'A1': { v: '', f: 'A2+A3', s: '' },
382
+
'A2': { v: 100, f: '', s: '' },
383
+
'A3': { v: 200, f: '', s: '' },
384
+
});
385
+
386
+
deleteRow(getCells, setCellData, 2, 3);
387
+
388
+
// A1's formula referenced A2 (now deleted) and A3 (now A2)
389
+
expect(getCellData('A1')?.f).toBe('#REF!+A2');
390
+
});
391
+
392
+
it('shifts formulas in cells below the deleted row up', () => {
393
+
const { getCells, setCellData, getCellData } = createMockCellMap({
394
+
'A1': { v: 10, f: '', s: '' },
395
+
'A2': { v: 20, f: '', s: '' },
396
+
'A3': { v: '', f: 'A1+A2', s: '' },
397
+
});
398
+
399
+
// Delete row 2
400
+
deleteRow(getCells, setCellData, 2, 3);
401
+
402
+
// A3 moves to A2, formula A1+A2 -> A1 stays, A2 was deleted = #REF!
403
+
expect(getCellData('A2')?.f).toBe('A1+#REF!');
404
+
});
405
+
});
406
+
407
+
// ============================================================
408
+
// insertColumn — integration with mock cell map
409
+
// ============================================================
410
+
411
+
describe('insertColumn', () => {
412
+
it('shifts cells right and adjusts formulas', () => {
413
+
const { getCells, setCellData, getCellData } = createMockCellMap({
414
+
'A1': { v: 10, f: '', s: '' },
415
+
'B1': { v: 20, f: '', s: '' },
416
+
'C1': { v: '', f: 'A1+B1', s: '' },
417
+
});
418
+
419
+
insertColumn(getCells, setCellData, 2, 1);
420
+
421
+
expect(getCellData('A1')?.v).toBe(10);
422
+
expect(getCellData('C1')?.v).toBe(20);
423
+
expect(getCellData('D1')?.f).toBe('A1+C1');
424
+
expect(getCellData('B1')).toBeNull();
425
+
});
426
+
427
+
it('does not affect cells left of the insert point', () => {
428
+
const { getCells, setCellData, getCellData } = createMockCellMap({
429
+
'A1': { v: 'stay', f: '', s: '' },
430
+
'D1': { v: 'move', f: '', s: '' },
431
+
});
432
+
433
+
insertColumn(getCells, setCellData, 3, 1);
434
+
435
+
expect(getCellData('A1')?.v).toBe('stay');
436
+
expect(getCellData('E1')?.v).toBe('move');
437
+
});
438
+
});
439
+
440
+
// ============================================================
441
+
// deleteColumn — integration with mock cell map
442
+
// ============================================================
443
+
444
+
describe('deleteColumn', () => {
445
+
it('removes the target column and shifts cells left', () => {
446
+
const { getCells, setCellData, getCellData } = createMockCellMap({
447
+
'A1': { v: 10, f: '', s: '' },
448
+
'B1': { v: 'delete me', f: '', s: '' },
449
+
'C1': { v: 30, f: '', s: '' },
450
+
});
451
+
452
+
deleteColumn(getCells, setCellData, 2, 1);
453
+
454
+
expect(getCellData('A1')?.v).toBe(10);
455
+
expect(getCellData('B1')?.v).toBe(30);
456
+
expect(getCellData('C1')).toBeNull();
457
+
});
458
+
459
+
it('adjusts formulas that referenced the deleted column to #REF!', () => {
460
+
const { getCells, setCellData, getCellData } = createMockCellMap({
461
+
'A1': { v: '', f: 'B1+C1', s: '' },
462
+
'B1': { v: 100, f: '', s: '' },
463
+
'C1': { v: 200, f: '', s: '' },
464
+
});
465
+
466
+
deleteColumn(getCells, setCellData, 2, 1);
467
+
468
+
expect(getCellData('A1')?.f).toBe('#REF!+B1');
469
+
});
470
+
});
471
+
472
+
// ============================================================
473
+
// Preserves cell styles through operations
474
+
// ============================================================
475
+
476
+
describe('style preservation', () => {
477
+
it('preserves cell styles when inserting a row', () => {
478
+
const { getCells, setCellData, getCellData } = createMockCellMap({
479
+
'A2': { v: 'styled', f: '', s: JSON.stringify({ bold: true, color: '#ff0000' }) },
480
+
});
481
+
482
+
insertRow(getCells, setCellData, 2, 1);
483
+
484
+
const moved = getCellData('A3');
485
+
expect(moved).not.toBeNull();
486
+
expect(moved!.v).toBe('styled');
487
+
// Style should be preserved (may be double-serialized depending on path)
488
+
const parsedStyle = typeof moved!.s === 'string' ? JSON.parse(moved!.s as string) : moved!.s;
489
+
expect(parsedStyle.bold).toBe(true);
490
+
});
491
+
492
+
it('preserves cell styles when inserting a column', () => {
493
+
const { getCells, setCellData, getCellData } = createMockCellMap({
494
+
'B1': { v: 'styled', f: '', s: JSON.stringify({ italic: true }) },
495
+
});
496
+
497
+
insertColumn(getCells, setCellData, 2, 1);
498
+
499
+
const moved = getCellData('C1');
500
+
expect(moved).not.toBeNull();
501
+
expect(moved!.v).toBe('styled');
502
+
});
503
+
});