[Mobile] Budget table revamp #2642
Conversation
github-actions
bot
changed the title
✅ Deploy Preview for actualbudget ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
trafico-bot
bot
added
🚧 WIP
Bundle Stats — desktop-clientHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger
Smaller
Unchanged
|
Bundle Stats — loot-coreHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded No assets were added Removed No assets were removed Bigger No assets were bigger Smaller
Unchanged No assets were unchanged |
|
I think this doesn't work very well with bigger amounts in the budget (decimals do not show). PR:
Current state:
|
joel-jeremy
force-pushed
the
replace-underline-with-pills
branch
2 times, most recently
from
dead25a
to
43a7a6f
Compare
joel-jeremy
force-pushed
the
replace-underline-with-pills
branch
2 times, most recently
from
e0d1ea1
to
f4e88d2
Compare
joel-jeremy
force-pushed
the
replace-underline-with-pills
branch
from
4ea0e0d
to
4ffe0e3
Compare
joel-jeremy
changed the title
trafico-bot
bot
added
🔍 Ready for Review
|
@jsehnoutka Thanks for the feedback! Pushed some changes to resize text when amount gets too long |
|
@joel-jeremy Looks great now, thanks for your work! Would it be possible to leave the same amount of characters available for the category name though? Maybe put the new > UI element a little bit to the right and assign fixed position to it? There are now less characters available for the category name before the line breaks, also the > UI element does not look very streamlined because it shows on different positions depending on the category name length. I think it would look better if it had consistent place on each line (category), nevertheless the text lenght. Before:
After:
|
|
Agree, category chevron would look best aligned with the group chevron. Not too sure about it changing alignment between groups, but when aligned with the group one you'll know the position will always work for the categories within the group as the group level totals the values underneath so will always have the widest numbers. |
|
Something like this? |
Yeah, it removes the higgilitypiggilty (I'm assuming the blank space is part of the tapable area). |
|
Pushed an update
Yes it's tapable |
joel-jeremy
requested review from
MatissJanis,
youngcw,
Teprifer,
shall0pass and
carkom
|
Looks great now, but still kills a lot of the characters available for category naming. If there is any possibility to leave this in the state it was before the PR, that would be great.
|
|
I not sure what I think about the lined up category arrows. Im fine with all the other changes though |
|
I'm also torn on lining up the cheverons on the group/category names. Lining them up looks good visually but also kinda wastes some space. I'm a leaning a bit towards not lining them up since that's how the desktop does it in the desktop budget table. More feedback would be good here. |
There was a problem hiding this comment.
Any idea why this would be changing?
joel-jeremy
force-pushed
the
replace-underline-with-pills
branch
from
48570c2
to
af81b52
Compare
|
Why was it changed to use a chevron on the right? To me, this makes it a bit confusing because now that chevron is being used for two purposes: 
Here, the left chevron means you can expand the group. The right one means you can pop up a dialog. |
|
And I guess this is just a bigger design question, as now there are chevrons all over and that apparently means "you can click on this". But it does somewhat confuse the viewer because chevrons mean "expand this" in some contexts. The old style with underlines was clear: underlined text are links, you can click on those. |
|
One more note:
Other than that this looks really good! |
joel-jeremy
force-pushed
the
replace-underline-with-pills
branch
from
6bac9e1
to
b1ceab8
Compare
No description provided.