Skip to content

Add column sorting to cost basis table - #24

Merged
mcintyre94 merged 1 commit into
mainfrom
add-cost-basis-table-sorting
Feb 8, 2026
Merged

Add column sorting to cost basis table#24
mcintyre94 merged 1 commit into
mainfrom
add-cost-basis-table-sorting

Conversation

@mcintyre94

@mcintyre94 mcintyre94 commented Feb 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Adds clickable sort headers for Token (by name), Cost Basis, Current Price, and Change columns
  • Click a column to sort ascending, click again to toggle to descending
  • Sort indicators show current sort state (arrow up/down) and sortable columns (up-down icon)
  • Entries with missing price/change data are pushed to the bottom regardless of sort direction

Test plan

  • Verify clicking each column header sorts the table correctly
  • Verify clicking the same header again reverses the sort direction
  • Verify sort icons update to reflect the active sort column and direction
  • Verify entries without current price data sort to the bottom
  • Verify the Actions column is not sortable

🤖 Generated with Claude Code

Adds clickable sort headers for Token (by name), Cost Basis, Current Price,
and Change columns. Click a column to sort ascending, click again to toggle
descending. Entries with missing price data are pushed to the bottom.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Feb 8, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
swapsies Ready Ready Preview, Comment Feb 8, 2026 6:51pm

@claude

claude Bot commented Feb 8, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@mcintyre94
mcintyre94 merged commit 32b614f into main Feb 8, 2026
4 checks passed
@mcintyre94
mcintyre94 deleted the add-cost-basis-table-sorting branch February 8, 2026 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant