Skip to content

improvement(tables): tables multi-select, keyboard shortcuts, and docs #3615

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
waleedlatif1 merged 11 commits into from
Mar 17, 2026
Merged

improvement(tables): tables multi-select, keyboard shortcuts, and docs #3615

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
11fce3b
added docs for tables
waleedlatif1 Mar 16, 2026
3e4241c
fix: remove generated files and move image to correct path
waleedlatif1 Mar 16, 2026
f326b05
fix(tables): skip gap positions in handleCopy for checked rows
waleedlatif1 Mar 16, 2026
e25770b
fix(tables): remove dead props from DataRow and PositionGapRows
waleedlatif1 Mar 17, 2026
116648b
fix(tables): clear range selection on Shift+Space to prevent mixed state
waleedlatif1 Mar 17, 2026
4ea4490
fix(tables): remove dead handleRowMouseDown and handleRowMouseEnter
waleedlatif1 Mar 17, 2026
29c765c
fix(tables): move Escape and Cmd+A above anchor guard
waleedlatif1 Mar 17, 2026
3e51e1c
fix(tables): Shift+Space preserves anchor for continued keyboard use
waleedlatif1 Mar 17, 2026
ad34616
fix(tables): only add real row positions in shift-click range select
waleedlatif1 Mar 17, 2026
57d7f76
fix(tables): Delete/Backspace works after Cmd+A select all
waleedlatif1 Mar 17, 2026
4c16333
docs(tables): clarify Delete/Backspace clears all columns in checkbox…
waleedlatif1 Mar 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix(tables): only add real row positions in shift-click range select 
Previously the loop added every integer from..to, which in sparse
tables (with deleted rows) could add thousands of phantom gap positions.
Now iterates positionMap to only add positions with actual rows.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
  • Loading branch information
@waleedlatif1 @claude
waleedlatif1 and claude committed Mar 17, 2026
commit ad3461656ea9fb20fa66e8fcb40ddee58b9cc932
5 changes: 3 additions & 2 deletions apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table/table.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -539,10 +539,11 @@ export function Table({
if (shiftKey && lastCheckboxRowRef.current !== null) {
const from = Math.min(lastCheckboxRowRef.current, rowIndex)
const to = Math.max(lastCheckboxRowRef.current, rowIndex)
const pMap = positionMapRef.current
setCheckedRows((prev) => {
const next = new Set(prev)
for (let i = from; i <= to; i++) {
next.add(i)
for (const [pos] of pMap) {
if (pos >= from && pos <= to) next.add(pos)
}
return next
})
Comment thread
waleedlatif1 marked this conversation as resolved.
Expand Down
Loading