Skip to content

fix: align daily net total to right screen edge - #155

Closed
Muthuraj5107220 wants to merge 1 commit into
PATILYASHH:masterfrom
Muthuraj5107220:fix/day-header-alignment-transactions
Closed

Muthuraj5107220 wants to merge 1 commit into
PATILYASHH:masterfrom
Muthuraj5107220:fix/day-header-alignment-transactions

Conversation

@Muthuraj5107220

Copy link
Copy Markdown

Description

Fixed layout issue in _DayHeader where the daily net amount failed to remain right-aligned.

Closes #154

Changes

-MainAxisAlignment.spaceBetween on Parent Row: Forces the left group and the right MoneyText to opposite ends of the container.

-Left Grouping into Nested Row: Wraps icon, date label, and count chip into a single flexible Row(mainAxisSize: MainAxisSize.min). This prevents the date label from pushing or shifting the right-aligned money text when dates are short or long.

Money-honesty checklist

  • No double used for money — amounts stay integer paise
  • Transfers / person movements still excluded from income, expense & budgets
  • Nothing posts to the ledger without user confirmation
  • Schema change? schemaVersion bumped + migration + old-backup restore test

Verification

  • flutter analyze with 1 warning(s)
  • flutter test green (new behavior has new tests)

@PATILYASHH

PATILYASHH commented Oct 8, 2026 •

Copy link
Copy Markdown
Owner

Can you please send a screenshot of transaction page

@Muthuraj5107220

Copy link
Copy Markdown
Author

Screenshot_2026-10-08-11-37-04-79_8ec1b01c745c2249339130d1103ced69.jpg

@PATILYASHH

Copy link
Copy Markdown
Owner

Should we change that red and green colors to yellow and orange because it will be easy to identify. What do you think ??

@Muthuraj5107220

Copy link
Copy Markdown
Author

Should we change that red and green colors to yellow and orange because it will be easy to identify. What do you think ??

Not really. But we can make it optional like we can make a option to choose colour by the user in settings section. I know it is a big development but making it as with the user makes the application more custamizable.

@PATILYASHH

Copy link
Copy Markdown
Owner

can you code for this feature ?? plz

@Muthuraj5107220

Copy link
Copy Markdown
Author

can you code for this feature ?? plz

Actually, I am a .NET developer not a Flutter developer. I am an open source activist so I just want to be a part open source application and I planned to create my own expense manager and this application is same as what I want so I just had changed that alignment issue using AI buy I don't have a great knowledge in Flutter.

I will try my best to create that if possible.

@PATILYASHH

Copy link
Copy Markdown
Owner

I think current place is good because if we make day total inline with transaction amounts it can cause confusion

@PATILYASHH PATILYASHH closed this Oct 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug: Day total alignment issue

2 participants