[PR #1563] [MERGED] 🐛 Link Schedules modal list of schedules grows too long #3821

Closed
opened 2026-02-28 20:47:17 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1563
Author: @trevdor
Created: 8/22/2023
Status: Merged
Merged: 8/22/2023
Merged by: @trevdor

Base: masterHead: trevdor-schedules-layout-fix


📝 Commits (2)

  • 3d729be 🐛 Link Schedules modal list of schedules grows too long
  • e4b8994 Release notes

📊 Changes

3 files changed (+11 additions, -4 deletions)

View changed files

📝 packages/desktop-client/src/components/schedules/LinkSchedule.tsx (+4 -2)
📝 packages/desktop-client/src/components/schedules/index.js (+1 -2)
upcoming-release-notes/1563.md (+6 -0)

📄 Description

Follow-up for #1501. The Schedules list height needs an upper bound as observed here.

Testing Notes

  1. Check the Schedules screen with no schedules. Should show the "No schedules" text and have this empty height:
    Screenshot 2023-08-22 at 9 28 33 AM

  2. Check the Link Schedule modal. Even with many schedules, the list should be capped at a reasonable length, like so:
    Screenshot 2023-08-22 at 9 27 41 AM

Demo budget file with many schedules: 2023-08-22-_test-budget.zip


🔄 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/1563 **Author:** [@trevdor](https://github.com/trevdor) **Created:** 8/22/2023 **Status:** ✅ Merged **Merged:** 8/22/2023 **Merged by:** [@trevdor](https://github.com/trevdor) **Base:** `master` ← **Head:** `trevdor-schedules-layout-fix` --- ### 📝 Commits (2) - [`3d729be`](https://github.com/actualbudget/actual/commit/3d729be5a0cc82944706bc798b68f5fd7c2e1e4f) :bug: Link Schedules modal list of schedules grows too long - [`e4b8994`](https://github.com/actualbudget/actual/commit/e4b89942a06cf64ac8a2c549af6a0a40576e3147) Release notes ### 📊 Changes **3 files changed** (+11 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/schedules/LinkSchedule.tsx` (+4 -2) 📝 `packages/desktop-client/src/components/schedules/index.js` (+1 -2) ➕ `upcoming-release-notes/1563.md` (+6 -0) </details> ### 📄 Description Follow-up for #1501. The Schedules list height needs an upper bound as [observed here](https://github.com/actualbudget/actual/pull/1501#issuecomment-1686520747). ### Testing Notes 1. Check the Schedules screen with no schedules. Should show the "No schedules" text and have this empty height: <img width="628" alt="Screenshot 2023-08-22 at 9 28 33 AM" src="https://github.com/actualbudget/actual/assets/5862724/e12c3392-9e6b-4897-8bf1-9e0f9ff319d5"> 2. Check the Link Schedule modal. Even with many schedules, the list should be capped at a reasonable length, like so: <img width="712" alt="Screenshot 2023-08-22 at 9 27 41 AM" src="https://github.com/actualbudget/actual/assets/5862724/7923f09a-adc9-4e2c-9d4f-5cb26b621b60"> Demo budget file with many schedules: [2023-08-22-_test-budget.zip](https://github.com/actualbudget/actual/files/12410320/2023-08-22-_test-budget.zip) --- <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-02-28 20:47:17 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#3821