[PR #4097] [MERGED] Do not show undo notifications on desktop #5186

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/4097
Author: @psybers
Created: 1/6/2025
Status: Merged
Merged: 1/7/2025
Merged by: @youngcw

Base: masterHead: no-undonotif-desktop


📝 Commits (9)

  • c6b1f2a Do not show undo notifications on desktop
  • 24f4391 add release note
  • bcdc82d fix linter
  • 5eb476d Update packages/desktop-client/src/hooks/useUndo.ts
  • b08c54c Update useUndo.ts
  • b7c41ea fix code pasted on wrong line
  • 05cc9a8 drive-by fix typo
  • ffbeab7 Update 4097.md
  • b0c82b9 Merge branch 'master' into no-undonotif-desktop

📊 Changes

2 files changed (+21 additions, -4 deletions)

View changed files

📝 packages/desktop-client/src/hooks/useUndo.ts (+15 -4)
upcoming-release-notes/4097.md (+6 -0)

📄 Description

Undo notifications (e.g. when doing actions from the budget menus) state what just happened and provide a button to easily undo that action. This is mostly only useful on mobile, since desktop users can just CTRL-Z any time. This change makes those notifications only show on narrow (mobile) layouts.


🔄 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/4097 **Author:** [@psybers](https://github.com/psybers) **Created:** 1/6/2025 **Status:** ✅ Merged **Merged:** 1/7/2025 **Merged by:** [@youngcw](https://github.com/youngcw) **Base:** `master` ← **Head:** `no-undonotif-desktop` --- ### 📝 Commits (9) - [`c6b1f2a`](https://github.com/actualbudget/actual/commit/c6b1f2aaf4ffe3ce1dcbf33a2a6603489b967ae4) Do not show undo notifications on desktop - [`24f4391`](https://github.com/actualbudget/actual/commit/24f4391eb1c4fa3690e437cbf9d900992909ec0c) add release note - [`bcdc82d`](https://github.com/actualbudget/actual/commit/bcdc82d57d43adad030187ebc44c4b74603208ef) fix linter - [`5eb476d`](https://github.com/actualbudget/actual/commit/5eb476dc63c138be7e8910af773504fb31a9c1ad) Update packages/desktop-client/src/hooks/useUndo.ts - [`b08c54c`](https://github.com/actualbudget/actual/commit/b08c54c26c7e9858f972d97771b912cc3a73ca34) Update useUndo.ts - [`b7c41ea`](https://github.com/actualbudget/actual/commit/b7c41ea403137292f4df65c39ace273ded4d45ba) fix code pasted on wrong line - [`05cc9a8`](https://github.com/actualbudget/actual/commit/05cc9a8763ece6186065e0bd8644282337e4bd95) drive-by fix typo - [`ffbeab7`](https://github.com/actualbudget/actual/commit/ffbeab7402feb37ac0f78e22b3e80562672ebc0d) Update 4097.md - [`b0c82b9`](https://github.com/actualbudget/actual/commit/b0c82b99bf90e6af803e7d88edcf04069d6f14d4) Merge branch 'master' into no-undonotif-desktop ### 📊 Changes **2 files changed** (+21 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/hooks/useUndo.ts` (+15 -4) ➕ `upcoming-release-notes/4097.md` (+6 -0) </details> ### 📄 Description Undo notifications (e.g. when doing actions from the budget menus) state what just happened and provide a button to easily undo that action. This is mostly only useful on mobile, since desktop users can just CTRL-Z any time. This change makes those notifications only show on narrow (mobile) layouts. --- <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:07:37 -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#5186