fix(#44): pin expense filter chips directly under tabs #48

Merged
admin merged 1 commits from fix/44-expenses-screen-empty-gap-tabs-chips into main 2026-06-28 14:58:58 +00:00
Owner

Summary

  • Hoist the date-range filter chip group and divider out of the LazyColumn in ExpensesTabContent and into a Column wrapper, so the chips sit directly under the Expenses/Recurring tabs and the list fills the rest of the page.
  • Add Modifier.fillMaxSize() to the Recurring tab's LazyColumn so it fills the pager page (no chips to hoist, but the layout is now consistent with the Expenses tab).

Test plan

  • ./gradlew assembleDebug — compiles
  • ./gradlew testDebugUnitTest — all unit tests pass
  • Manual: open the Expenses screen with at least one expense, confirm filter chips are right under the tabs and the list starts immediately below them. Repeat for the Recurring tab.
  • Manual: switch tabs to confirm no regression in tab switching or scrolling.

Closes #44

## Summary - Hoist the date-range filter chip group and divider out of the `LazyColumn` in `ExpensesTabContent` and into a `Column` wrapper, so the chips sit directly under the Expenses/Recurring tabs and the list fills the rest of the page. - Add `Modifier.fillMaxSize()` to the Recurring tab's `LazyColumn` so it fills the pager page (no chips to hoist, but the layout is now consistent with the Expenses tab). ## Test plan - [x] `./gradlew assembleDebug` — compiles - [x] `./gradlew testDebugUnitTest` — all unit tests pass - [ ] Manual: open the Expenses screen with at least one expense, confirm filter chips are right under the tabs and the list starts immediately below them. Repeat for the Recurring tab. - [ ] Manual: switch tabs to confirm no regression in tab switching or scrolling. Closes #44
admin added 1 commit 2026-06-28 14:57:25 +00:00
Hoist the date-range filter chip group and divider out of the
LazyColumn and into a Column wrapper, so the chips sit directly
under the Expenses/Recurring tabs and the list fills the rest of
the page. Previously the chips were the first item of the
LazyColumn, which left a large empty area between the tabs and
the chips.

Also add Modifier.fillMaxSize() to the Recurring tab's LazyColumn
so it fills the pager page (no chips to hoist, but the layout
is now consistent with the Expenses tab).
admin merged commit f6ac2aace1 into main 2026-06-28 14:58:58 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: admin/ledgerr#48