[PR #1531] [MERGED] 🐛 (darkmode) fix transaction list hover effects #3799

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/1531
Author: @MatissJanis
Created: 8/15/2023
Status: Merged
Merged: 8/16/2023
Merged by: @MatissJanis

Base: masterHead: matiss/fix-1498


📝 Commits (6)

  • 9327890 🐛 (darkmode) fix transaction list hover effects
  • b2e1964 Release notes
  • 2dc3ba1 Schedule hover color fix
  • cea461b Feedback fixes
  • 80e7b3c Merge branch 'master' into matiss/fix-1498
  • 0078496 Feedback: fix 'budget' column hover bg

📊 Changes

11 files changed (+52 additions, -179 deletions)

View changed files

📝 packages/desktop-client/src/components/budget/misc.js (+3 -4)
📝 packages/desktop-client/src/components/budget/report/components.tsx (+4 -17)
📝 packages/desktop-client/src/components/budget/rollover/rollover-components.tsx (+4 -19)
📝 packages/desktop-client/src/components/payees/index.js (+9 -10)
📝 packages/desktop-client/src/components/schedules/DiscoverSchedules.js (+1 -2)
📝 packages/desktop-client/src/components/schedules/SchedulesTable.js (+0 -1)
📝 packages/desktop-client/src/components/table.tsx (+24 -121)
📝 packages/desktop-client/src/components/transactions/TransactionList.js (+0 -2)
📝 packages/desktop-client/src/components/transactions/TransactionsTable.js (+0 -2)
📝 packages/desktop-client/src/style/themes/light.ts (+1 -1)
upcoming-release-notes/1531.md (+6 -0)

📄 Description

Closes #1498

Fixing the transaction table hover effects. Also removing animation code as this does not seem to be working anymore (and hasn't been working for a while..). So I think it's worthwhile to clean it up to reduce the complexity in the transaction table.


🔄 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/1531 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 8/15/2023 **Status:** ✅ Merged **Merged:** 8/16/2023 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/fix-1498` --- ### 📝 Commits (6) - [`9327890`](https://github.com/actualbudget/actual/commit/9327890885741868344f5dbfe55fa01b49c5db16) :bug: (darkmode) fix transaction list hover effects - [`b2e1964`](https://github.com/actualbudget/actual/commit/b2e1964f16d718d31175e24d75b0855f36768ee8) Release notes - [`2dc3ba1`](https://github.com/actualbudget/actual/commit/2dc3ba1e4f12e8992786e0a207199d997f857026) Schedule hover color fix - [`cea461b`](https://github.com/actualbudget/actual/commit/cea461b47e0f436c7a2f3fe04619fb9db311d37c) Feedback fixes - [`80e7b3c`](https://github.com/actualbudget/actual/commit/80e7b3c9538c1b639bf61eb975ade51e18068c1f) Merge branch 'master' into matiss/fix-1498 - [`0078496`](https://github.com/actualbudget/actual/commit/0078496162c54d56c6c87bc93af89063c01b5a3c) Feedback: fix 'budget' column hover bg ### 📊 Changes **11 files changed** (+52 additions, -179 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/budget/misc.js` (+3 -4) 📝 `packages/desktop-client/src/components/budget/report/components.tsx` (+4 -17) 📝 `packages/desktop-client/src/components/budget/rollover/rollover-components.tsx` (+4 -19) 📝 `packages/desktop-client/src/components/payees/index.js` (+9 -10) 📝 `packages/desktop-client/src/components/schedules/DiscoverSchedules.js` (+1 -2) 📝 `packages/desktop-client/src/components/schedules/SchedulesTable.js` (+0 -1) 📝 `packages/desktop-client/src/components/table.tsx` (+24 -121) 📝 `packages/desktop-client/src/components/transactions/TransactionList.js` (+0 -2) 📝 `packages/desktop-client/src/components/transactions/TransactionsTable.js` (+0 -2) 📝 `packages/desktop-client/src/style/themes/light.ts` (+1 -1) ➕ `upcoming-release-notes/1531.md` (+6 -0) </details> ### 📄 Description Closes #1498 Fixing the transaction table hover effects. Also removing `animation` code as this does not seem to be working anymore (and hasn't been working for a while..). So I think it's worthwhile to clean it up to reduce the complexity in the transaction table. --- <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:46:58 -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#3799