[PR #2773] [MERGED] Fix notes icon changing size/position with long category or group names #38934

Closed
opened 2026-04-23 11:54:22 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/2773
Author: @jpelgrom
Created: 5/17/2024
Status: Merged
Merged: 5/21/2024
Merged by: @joel-jeremy

Base: masterHead: fix-category-name-notes-icon-size


📝 Commits (1)

  • 5b91e9c Fix notes icon changing size/position with long category or group names

📊 Changes

3 files changed (+20 additions, -10 deletions)

View changed files

📝 packages/desktop-client/src/components/budget/SidebarCategory.tsx (+7 -5)
📝 packages/desktop-client/src/components/budget/SidebarGroup.tsx (+7 -5)
upcoming-release-notes/2773.md (+6 -0)

📄 Description

Fixes #2736 by telling the browser this icon should not be shrunk in these rows (default is 1, which means the other icons which have flexShrink: 0 get more space).

Before/after:

Before (demo.actualbudget.org) After (this branch)
image image
image image

(first contribution, so please let me know if I missed anything)


🔄 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/2773 **Author:** [@jpelgrom](https://github.com/jpelgrom) **Created:** 5/17/2024 **Status:** ✅ Merged **Merged:** 5/21/2024 **Merged by:** [@joel-jeremy](https://github.com/joel-jeremy) **Base:** `master` ← **Head:** `fix-category-name-notes-icon-size` --- ### 📝 Commits (1) - [`5b91e9c`](https://github.com/actualbudget/actual/commit/5b91e9cdb6542728121567e9946bf34c864afd6f) Fix notes icon changing size/position with long category or group names ### 📊 Changes **3 files changed** (+20 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/budget/SidebarCategory.tsx` (+7 -5) 📝 `packages/desktop-client/src/components/budget/SidebarGroup.tsx` (+7 -5) ➕ `upcoming-release-notes/2773.md` (+6 -0) </details> ### 📄 Description <!-- Thank you for submitting a pull request! Make sure to follow the instructions to write release notes for your PR — it should only take a minute or two: https://github.com/actualbudget/docs#writing-good-release-notes --> Fixes #2736 by telling the browser this icon should not be shrunk in these rows (default is 1, which means the other icons which have `flexShrink: 0` get more space). Before/after: |Before (demo.actualbudget.org)|After (this branch)| |----|----| |![image](https://github.com/actualbudget/actual/assets/8148535/04783556-facf-4117-aa0b-ce81839c9c57)|![image](https://github.com/actualbudget/actual/assets/8148535/bcf7236a-7e60-473e-b92e-75612ab5b538)| |![image](https://github.com/actualbudget/actual/assets/8148535/fdee8ac7-f8de-4903-9bdb-84f22b98215a)|![image](https://github.com/actualbudget/actual/assets/8148535/014ac0f7-1453-497f-908f-80f26814b340)| (first contribution, so please let me know if I missed anything) --- <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-23 11:54:22 -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#38934