Skip to content

feat: implement interactive bidirectional sorting columns#458

Merged
emdevelopa merged 1 commit into
emdevelopa:mainfrom
OperaCode:feat/advanced-sortable-table-headers
Mar 28, 2026
Merged

feat: implement interactive bidirectional sorting columns#458
emdevelopa merged 1 commit into
emdevelopa:mainfrom
OperaCode:feat/advanced-sortable-table-headers

Conversation

@OperaCode
Copy link
Copy Markdown
Contributor

Summary

  • add clickable sortable headers for Status, Amount, Recipient, and Date in the payments table
  • persist sortColumn and sortDirection in the URL so sorting survives refreshes and shared links
  • sort payment rows client-side while keeping the existing filter flow intact
  • add Playwright coverage for payments sorting and update dashboard test fixtures for the new recipient column

Closes #437

The payments table headers were static, which made it harder to scan and compare payments by amount, recency, status, or recipient.

What changed

  • updated the recent payments table to make all key columns interactive sort controls
  • added active sort direction indicators and aria-sort state to the table headers
  • replaced the description column in the table with the requested recipient column
  • kept sort state in the query string using sortColumn and sortDirection
  • added an e2e test covering default sort order, header toggling, and persisted query params
  • updated the visual regression payments fixture to include recipient values

@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 28, 2026

@OperaCode is attempting to deploy a commit to the Emmanuel's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Mar 28, 2026

@OperaCode Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@emdevelopa emdevelopa merged commit 8a0ba22 into emdevelopa:main Mar 28, 2026
2 of 3 checks passed
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.

[FE] Advanced Sortable Table Headers

2 participants