[PR #435] [MERGED] Read-only Responsive view #33328

Closed
opened 2026-04-20 20:43:04 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/435
Author: @trevdor
Created: 1/6/2023
Status: Merged
Merged: 1/10/2023
Merged by: @trevdor

Base: masterHead: trevdor-responsive


📝 Commits (10+)

  • 22c8c2e Read-only Responsive Web App (#79)
  • a32bba8 Merge branch 'master' into trevdor-responsive
  • 00e764c Merge branch 'master' into trevdor-responsive
  • 97224c7 fix: move AnimatedLoading path from svg to v1
  • 2fa14af Responding to tester feedback
  • 0bb1c67 Merge remote-tracking branch 'origin/master' into trevdor-responsive
  • b9e8f80 Split the Settings component into multiple files (#434)
  • 8b4946a PR feedback: Offline color on mobile
  • 786f4b3 Fix console warnings
  • 1b7cb31 Remove need for isMobile in CSS: lean on media queries in styles.js and glamor

📊 Changes

71 files changed (+4436 additions, -664 deletions)

View changed files

📝 packages/desktop-client/package.json (+9 -1)
📝 packages/desktop-client/public/index.html (+4 -1)
📝 packages/desktop-client/src/components/BankSyncStatus.js (+1 -1)
📝 packages/desktop-client/src/components/FinancesApp.js (+143 -45)
📝 packages/desktop-client/src/components/MobileWebMessage.js (+56 -32)
📝 packages/desktop-client/src/components/Modals.js (+10 -0)
📝 packages/desktop-client/src/components/Notifications.js (+1 -1)
packages/desktop-client/src/components/Settings.js (+0 -383)
packages/desktop-client/src/components/SyncRefresh.js (+13 -0)
📝 packages/desktop-client/src/components/Titlebar.js (+19 -6)
📝 packages/desktop-client/src/components/accounts/Account.js (+2 -2)
packages/desktop-client/src/components/accounts/MobileAccount.js (+287 -0)
packages/desktop-client/src/components/accounts/MobileAccountDetails.js (+196 -0)
packages/desktop-client/src/components/accounts/MobileAccounts.js (+352 -0)
packages/desktop-client/src/components/accounts/MobileTransaction.js (+498 -0)
packages/desktop-client/src/components/budget/MobileBudget.js (+313 -0)
packages/desktop-client/src/components/budget/MobileBudgetTable.js (+1204 -0)
packages/desktop-client/src/components/budget/MobileTable.js (+29 -0)
📝 packages/desktop-client/src/components/manager/ConfigServer.js (+3 -1)
📝 packages/desktop-client/src/components/manager/ManagementApp.js (+6 -5)

...and 51 more files

📄 Description

A PR will keep the build preview current for anyone testing.
👉🏻 Please report bugs, propose changes, etc. in discussion https://github.com/actualbudget/actual/discussions/433

Prior to releasing this particular version...

Awaiting Fix

Won't Fix

  1. Bah. I fixed the only won't-fix I had so far...

🔄 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/435 **Author:** [@trevdor](https://github.com/trevdor) **Created:** 1/6/2023 **Status:** ✅ Merged **Merged:** 1/10/2023 **Merged by:** [@trevdor](https://github.com/trevdor) **Base:** `master` ← **Head:** `trevdor-responsive` --- ### 📝 Commits (10+) - [`22c8c2e`](https://github.com/actualbudget/actual/commit/22c8c2e8f2be207376b1d1e24ce0b5a7e3534d87) Read-only Responsive Web App (#79) - [`a32bba8`](https://github.com/actualbudget/actual/commit/a32bba83d44c945fe955e1f8567e2d5cdaba2c7a) Merge branch 'master' into trevdor-responsive - [`00e764c`](https://github.com/actualbudget/actual/commit/00e764ccee873a7b3623ba146f05152e4eb804d8) Merge branch 'master' into trevdor-responsive - [`97224c7`](https://github.com/actualbudget/actual/commit/97224c7877d3af3c03b1790a0737ceeabf1ecbc1) fix: move AnimatedLoading path from svg to v1 - [`2fa14af`](https://github.com/actualbudget/actual/commit/2fa14afea44f208143b457f4f6d5f9ce7e8ca4f0) Responding to tester feedback - [`0bb1c67`](https://github.com/actualbudget/actual/commit/0bb1c67e58558f80386bb620146cbab3e18a5a23) Merge remote-tracking branch 'origin/master' into trevdor-responsive - [`b9e8f80`](https://github.com/actualbudget/actual/commit/b9e8f80a9cd4ca14db81e688be892d386ec53205) Split the Settings component into multiple files (#434) - [`8b4946a`](https://github.com/actualbudget/actual/commit/8b4946aa96736646d986563f699d81a672ca9fc5) PR feedback: Offline color on mobile - [`786f4b3`](https://github.com/actualbudget/actual/commit/786f4b3dd034940ead7cb1572e9687b575efa90b) Fix console warnings - [`1b7cb31`](https://github.com/actualbudget/actual/commit/1b7cb31bc09de86e693c30eddd7d4350a2717d79) Remove need for isMobile in CSS: lean on media queries in styles.js and glamor ### 📊 Changes **71 files changed** (+4436 additions, -664 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/package.json` (+9 -1) 📝 `packages/desktop-client/public/index.html` (+4 -1) 📝 `packages/desktop-client/src/components/BankSyncStatus.js` (+1 -1) 📝 `packages/desktop-client/src/components/FinancesApp.js` (+143 -45) 📝 `packages/desktop-client/src/components/MobileWebMessage.js` (+56 -32) 📝 `packages/desktop-client/src/components/Modals.js` (+10 -0) 📝 `packages/desktop-client/src/components/Notifications.js` (+1 -1) ➖ `packages/desktop-client/src/components/Settings.js` (+0 -383) ➕ `packages/desktop-client/src/components/SyncRefresh.js` (+13 -0) 📝 `packages/desktop-client/src/components/Titlebar.js` (+19 -6) 📝 `packages/desktop-client/src/components/accounts/Account.js` (+2 -2) ➕ `packages/desktop-client/src/components/accounts/MobileAccount.js` (+287 -0) ➕ `packages/desktop-client/src/components/accounts/MobileAccountDetails.js` (+196 -0) ➕ `packages/desktop-client/src/components/accounts/MobileAccounts.js` (+352 -0) ➕ `packages/desktop-client/src/components/accounts/MobileTransaction.js` (+498 -0) ➕ `packages/desktop-client/src/components/budget/MobileBudget.js` (+313 -0) ➕ `packages/desktop-client/src/components/budget/MobileBudgetTable.js` (+1204 -0) ➕ `packages/desktop-client/src/components/budget/MobileTable.js` (+29 -0) 📝 `packages/desktop-client/src/components/manager/ConfigServer.js` (+3 -1) 📝 `packages/desktop-client/src/components/manager/ManagementApp.js` (+6 -5) _...and 51 more files_ </details> ### 📄 Description A PR will keep the build preview current for anyone testing. 👉🏻 Please report bugs, propose changes, etc. in discussion https://github.com/actualbudget/actual/discussions/433 Prior to releasing this particular version... ### Awaiting Fix - [x] Settings screen layout issues on mobile * @j-f1 fixed in https://github.com/actualbudget/actual/pull/434 - [x] "Close budget" button broken on mobile * https://github.com/actualbudget/actual/discussions/433#discussioncomment-4602003 * @j-f1 fixed in https://github.com/actualbudget/actual/pull/434 - [x] Styles not recomputing correctly in some cases on window resize * also applies to mobile users rotating from portrait to landscape - [ ] UI fix ups - [x] Budget screen bottom margin * https://github.com/actualbudget/actual/discussions/433#discussioncomment-4603877 - [x] Budget screen Offline label color * https://github.com/actualbudget/actual/discussions/433#discussioncomment-4603885 - [x] Account view search: Empty results state * https://github.com/actualbudget/actual/discussions/433#discussioncomment-4603889 - [ ] Server config screen: overlap in footer text * https://github.com/actualbudget/actual/discussions/433#discussioncomment-4605317 - [x] Budget/Accounts/Settings header consistency * https://github.com/actualbudget/actual/discussions/433#discussioncomment-4603898 ### Won't Fix 0. Bah. I fixed the only won't-fix I had so far... --- <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 20:43:04 -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#33328