The CI dart analyze step runs with --fatal-infos so any
avoid_redundant_argument_values info fails the build. The new
EmailThreadListController test passed the default accountId explicitly,
tripping the lint. Use the default instead.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Closes#533.
Pull selection, swipe, pagination and batch actions out of three
near-duplicate screens (EmailListScreen, CombinedInboxScreen,
AddressEmailsScreen) into a single shared widget. Folder view, combined
inbox, in-folder search results and by-address lists now share one tile
renderer, one selection controller and one batch-action bottom bar.
- New EmailThreadList widget + EmailThreadListController own the
list rendering, selection set, optional swipe-to-archive/delete and
optional pagination. Hosts listen to the controller to swap between
their normal AppBar/drawer/FAB and the shared selection AppBar /
BottomAppBar (buildSelectionAppBar, buildSelectionBottomBar).
- Batch actions (batchArchive, batchDelete, batchMarkSpam,
batchMove, batchSnooze) and swipeDismissThread move to
email_action_helpers.dart and group threads by account so multi-
account selections produce correctly scoped repository calls and undo
actions. The combined inbox now supports the full action set (was
archive + delete only).
- The duplicate EmailThreadTile widget is removed; ThreadTile is the
single tile used everywhere. Search results now render with the same
unread/flag icons as the inbox list.
- AddressEmailsScreen adopts the shared list, gaining selection +
batch actions for free.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
