[PR #5984] [MERGED] make SelectLinkedAccountModal responsive #36449

Closed
opened 2026-04-20 23:18:40 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/5984
Author: @matt-fidd
Created: 10/22/2025
Status: Merged
Merged: 11/5/2025
Merged by: @matt-fidd

Base: masterHead: mobile/link-modal


📝 Commits (10+)

📊 Changes

2 files changed (+348 additions, -86 deletions)

View changed files

📝 packages/desktop-client/src/components/modals/SelectLinkedAccountsModal.tsx (+342 -86)
upcoming-release-notes/5984.md (+6 -0)

📄 Description

Follow up to https://github.com/actualbudget/actual/pull/5978 taking inspiration from @lelemm (https://github.com/actualbudget/actual/pull/5978#issuecomment-3430315257)

Also fixes the table headers that have been broken at least since 25.10 but possibly further back

Best to diff with whitespace off here to see the real changes.

Desktop:
image

Mobile:
image

Tested with:

  • Gocardless
  • SimpleFIN
  • Pluggy.ai

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/5984 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 10/22/2025 **Status:** ✅ Merged **Merged:** 11/5/2025 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `mobile/link-modal` --- ### 📝 Commits (10+) - [`fb2ec46`](https://github.com/actualbudget/actual/commit/fb2ec4698115932146f012e3b1806982516e518e) make SelectLinkedAccountModal responsive - [`7ebf687`](https://github.com/actualbudget/actual/commit/7ebf687a96b51ec52927a5f2b896d2645e7a2f5f) fix table headers - [`bfa8115`](https://github.com/actualbudget/actual/commit/bfa811545223f8f64f71b7ff26d18f27a817b126) note - [`d1d3e36`](https://github.com/actualbudget/actual/commit/d1d3e360f5ae12861490ba534cd62086becc23c0) coderabbit - [`465a3a3`](https://github.com/actualbudget/actual/commit/465a3a3fa51af8574684a93b10327b16f17abcb5) fix feedback - [`89cea7e`](https://github.com/actualbudget/actual/commit/89cea7ecc9b346d89a1543f4a0e570eff12d870d) coderabbit - [`62fa79e`](https://github.com/actualbudget/actual/commit/62fa79effc57d6049601431e0579773e04d41174) cast to string - [`19285ed`](https://github.com/actualbudget/actual/commit/19285ed5945c09e63feee2ccf0c63b16852ed081) Fix for unlinking account only - [`fe238db`](https://github.com/actualbudget/actual/commit/fe238db18ad72178123ae9cee467b7573d2c2690) Code Rabbit reviews - [`c9beca6`](https://github.com/actualbudget/actual/commit/c9beca64c811de25b95c8c32e1f37361f5faf852) Merge branch 'master' into mobile/link-modal ### 📊 Changes **2 files changed** (+348 additions, -86 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/modals/SelectLinkedAccountsModal.tsx` (+342 -86) ➕ `upcoming-release-notes/5984.md` (+6 -0) </details> ### 📄 Description Follow up to https://github.com/actualbudget/actual/pull/5978 taking inspiration from @lelemm (https://github.com/actualbudget/actual/pull/5978#issuecomment-3430315257) Also fixes the table headers that have been broken at least since 25.10 but possibly further back Best to diff with whitespace off here to see the real changes. Desktop: <img width="1000" height="448" alt="image" src="https://github.com/user-attachments/assets/f35d08ca-10bc-4ec8-ba1b-ff2e9c9504fb" /> Mobile: <img width="373" height="598" alt="image" src="https://github.com/user-attachments/assets/2b130f9a-d16d-455b-ab52-2abfed68db6a" /> Tested with: - [x] Gocardless - [x] SimpleFIN - [x] Pluggy.ai --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-20 23:18:40 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#36449