Refactor: Clean up BankTransactions.tsx conditional logic

[jimwei1]

Description

Refactor: Clean up BankTransactions.tsx conditional logic

Changes

• Separate height/width conditional logic
• Add ViewType type + clean up conditional logic

---

Note

Streamlines how bank transactions views are determined and rendered.

• Adds ViewType enum and viewType useMemo to drive Table/List/MobileList/Loading rendering in BankTransactions.tsx
• Simplifies container size handler: computes targetShift for sticky header and shouldBeListView from width; debounced updates
• Replaces scattered conditionals with clean viewType-based JSX branches, including unified Loader handling
• Minor import reorder for NewToggle in BankTransactionsHeader.tsx and mobile list item expanded row

^{Written by Cursor Bugbot for commit 7e03c52. This will update automatically on new commits. Configure here.}

Context used:

• Rule from dashboard - In Typescript files, use type declarations instead of interface declarations if the object is in... (source)

[augmentcode]

🤖 Augment PR Summary

Summary: Refactors BankTransactions.tsx to simplify responsive rendering and reduce nested conditionals.

Changes:

• Computes shiftStickyHeader via a single targetShift derived from container height.
• Derives listView from container width using a single shouldBeListView flag.
• Introduces a ViewType union and memoized viewType to centralize table/list/mobile/loading render branches.

Technical Notes: Keeps the same breakpoint and loading behavior while making the conditions easier to reason about.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. No suggestions at this time.

Comment augment review to trigger a new review at any time.

[jimwei1]

This should actually now be using ResponsiveContainer. Closing and updating ticket.