[PR #3288] [MERGED] (dashboards) text widget #4777

Closed
opened 2026-02-28 21:01:07 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/3288
Author: @MatissJanis
Created: 8/18/2024
Status: Merged
Merged: 9/3/2024
Merged by: @MatissJanis

Base: masterHead: matiss/text-widget


📝 Commits (10+)

  • 745f43d (dashboards) ability to rename all widgets
  • 4224099 typescript: migrate cards from jsx to tsx
  • 762526d Undo some unnecessary changes
  • 1ee29e1 Merge branch 'matiss/ts-cards' into matiss/rename-all-widgets
  • 76f7bcc Unify some name logic
  • beb160f A bit more logic simplification
  • 86578b1 (dashboards) text widget
  • d4bcfde Merge branch 'master' into matiss/text-widget
  • 138e494 Merge branch 'master' into matiss/text-widget
  • 09eb611 Cleanup the code

📊 Changes

7 files changed (+202 additions, -34 deletions)

View changed files

📝 packages/desktop-client/src/components/Notes.tsx (+3 -0)
📝 packages/desktop-client/src/components/reports/Overview.tsx (+41 -30)
📝 packages/desktop-client/src/components/reports/ReportCardName.tsx (+1 -3)
packages/desktop-client/src/components/reports/reports/MarkdownCard.tsx (+141 -0)
📝 packages/loot-core/src/server/dashboard/app.ts (+1 -0)
📝 packages/loot-core/src/types/models/dashboard.d.ts (+9 -1)
upcoming-release-notes/3288.md (+6 -0)

📄 Description

Depends on https://github.com/actualbudget/actual/pull/3284 and https://github.com/actualbudget/actual/pull/3289

https://github.com/user-attachments/assets/ca3100dd-c2e1-4ec8-a2da-4c7a1e1e63ce

Open question: should text widgets have a "name"/"title" as the other widgets? Or perhaps there is NO name and instead the name can be set by updating the content of the widget (thus allowing to remove the name entirely if the user wishes to do that).


🔄 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/3288 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 8/18/2024 **Status:** ✅ Merged **Merged:** 9/3/2024 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/text-widget` --- ### 📝 Commits (10+) - [`745f43d`](https://github.com/actualbudget/actual/commit/745f43d3da8f8cebbc7959696205ebc822f072ee) :sparkles: (dashboards) ability to rename all widgets - [`4224099`](https://github.com/actualbudget/actual/commit/422409966e84b03ae027aed42fd8ca31144fc101) typescript: migrate cards from jsx to tsx - [`762526d`](https://github.com/actualbudget/actual/commit/762526d2bc84ef42379228f6cebdcb5a4d2a3c41) Undo some unnecessary changes - [`1ee29e1`](https://github.com/actualbudget/actual/commit/1ee29e14cae318bfd94c17806082704a67a906b5) Merge branch 'matiss/ts-cards' into matiss/rename-all-widgets - [`76f7bcc`](https://github.com/actualbudget/actual/commit/76f7bccaa9a907de8ed0f4d7d727ff77d546151e) Unify some name logic - [`beb160f`](https://github.com/actualbudget/actual/commit/beb160f626985cff22a1430e5770031134825422) A bit more logic simplification - [`86578b1`](https://github.com/actualbudget/actual/commit/86578b135f1223cef75839aa0ce933460dc02e68) :sparkles: (dashboards) text widget - [`d4bcfde`](https://github.com/actualbudget/actual/commit/d4bcfde515a91d847f86637af12dbb54d86b3f18) Merge branch 'master' into matiss/text-widget - [`138e494`](https://github.com/actualbudget/actual/commit/138e4944903c6fbd5c58c9e65dd1a754b9d446dc) Merge branch 'master' into matiss/text-widget - [`09eb611`](https://github.com/actualbudget/actual/commit/09eb611e239a153ee3e5fa703d361f674014f23a) Cleanup the code ### 📊 Changes **7 files changed** (+202 additions, -34 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/Notes.tsx` (+3 -0) 📝 `packages/desktop-client/src/components/reports/Overview.tsx` (+41 -30) 📝 `packages/desktop-client/src/components/reports/ReportCardName.tsx` (+1 -3) ➕ `packages/desktop-client/src/components/reports/reports/MarkdownCard.tsx` (+141 -0) 📝 `packages/loot-core/src/server/dashboard/app.ts` (+1 -0) 📝 `packages/loot-core/src/types/models/dashboard.d.ts` (+9 -1) ➕ `upcoming-release-notes/3288.md` (+6 -0) </details> ### 📄 Description Depends on https://github.com/actualbudget/actual/pull/3284 and https://github.com/actualbudget/actual/pull/3289 https://github.com/user-attachments/assets/ca3100dd-c2e1-4ec8-a2da-4c7a1e1e63ce **Open question:** should text widgets have a "name"/"title" as the other widgets? Or perhaps there is NO name and instead the name can be set by updating the content of the widget (thus allowing to remove the name entirely if the user wishes to do that). --- <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 21:01:07 -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#4777