actualbudget / actual

A local-first personal finance app
https://actualbudget.org
MIT License
12.51k stars 954 forks source link

[Bug]: side-nav visual hierarchy #2944

Open MatissJanis opened 6 days ago

MatissJanis commented 6 days ago

Verified issue does not already exist?

What happened?

Screenshot 2024-06-26 at 17 31 14

The addition of the underline for account categories (introduced here https://github.com/actualbudget/actual/pull/2847) has created an issue: the visual hierarchy between accounts and top-level items has been broken.

Some potential solutions:

Where are you hosting Actual?

None

What browsers are you seeing the problem on?

No response

Operating System

None

ParthJohri commented 5 days ago

Hi @MatissJanis , Thank you for the bug clarification, I have made the following changes (screenshot attached), pleasre let me know if any further changes need to be done. If everything looks good, then I will create the PR for it.

Modifications

  1. Went on with your first suggestion of slightly Increaseing the font size for "off budget/on budget/all accounts" links and removed the underline. All Accounts has slightly more font size than off budget/on budget.
  2. Capitalised each word's first letter to put emphasis on the title.

Screenshots

navbar-heirarchy

Thank you!

youngcw commented 5 days ago

I like the font size increase and the added capitalization. I do still like the underline though if there is a way to keep it and not make the All Accounts header get lost

MatissJanis commented 5 days ago

Agreed with @youngcw on all things except the underline. IMO it stands out too much in the side-nav and thus creates unnecessary clutter.

We'll need a tie-breaker here on this topic :)

ParthJohri commented 5 days ago

@MatissJanis @youngcw How about this one?

  1. Using a solid border to show a parent hierarchy of all accounts.
  2. Using a dotted border to show for budget/ off budget.

navbar-heirarchy2

newtonfav commented 5 days ago

The dotted border doesn't look nice. Why not use a solid border with a very thin border-width? Additionally, remove the solid border for the All Accounts header and increase its font size.

ParthJohri commented 5 days ago

The dotted border doesn't look nice. Why not use a solid border with a very thin border-width? Additionally, remove the solid border for the All Accounts header and increase its font size.

@newtonfav Kind of like this? navbar-heirarchy3

youngcw commented 5 days ago

thats pretty nice.

ParthJohri commented 5 days ago

thats pretty nice.

If the above looks good, I can create a PR for it, please let me know if there is any other change to be made.

MatissJanis commented 5 days ago

"All accounts" is too large. None of these nav items should exceed in size the "Setting/Reports/etc" nav items.

How would it look in lightmode? I'm still on the fence for the underlines.

ParthJohri commented 5 days ago

"All accounts" is too large. None of these nav items should exceed in size the "Setting/Reports/etc" nav items.

How would it look in lightmode? I'm still on the fence for the underlines.

@MatissJanis

All Accounts

Screenshots for all the themes:-

Light Mode

Screenshot 2024-06-28 at 02 37 34

Dark Mode

Screenshot 2024-06-28 at 02 37 50

Midnight Mode

Screenshot 2024-06-28 at 02 38 01

Teprifer commented 5 days ago

Not sure about bullet points, the account names would also have to be indented a fair bit.

One idea put forward in discord was to only slightly indent the account names by 5px.

MatissJanis commented 4 days ago

Of all the revisions: I liked best the 1st one and second-best was the last one.

Happy to go with either of those options if people still feel like the underline is a good idea for the UX.

ParthJohri commented 4 days ago

Of all the revisions: I liked best the 1st one and second-best was the last one.

Happy to go with either of those options if people still feel like the underline is a good idea for the UX.

@MatissJanis @youngcw @Teprifer @newtonfav How does this look?

  1. Indentation of 5 pixels for accounts, sub-accounts.
  2. Same font size but capitalisation of each account word.
  3. Horizontal line for the accounts, excluding the All Accounts.

navbar-ui-change

MatissJanis commented 1 day ago

Comparatively I prefered the 1st version best. The horizontal indentation looks out of place.

ParthJohri commented 4 hours ago

Comparatively I prefered the 1st version best. The horizontal indentation looks out of place.

Thank you @MatissJanis for your suggestions, if others agree then I can create the PR for it.

youngcw commented 1 hour ago

@ParthJohri Go ahead and make a PR we can keep discussing there if needed