fix: tapping account navigates to mailbox list
Tapping an account tile now opens the folder list rather than jumping straight to INBOX. The redundant "All mailboxes" popup entry is removed. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
co-authored by
Claude Sonnet 4.6
parent
9d2c297894
commit
bdc9d3cd5a
@@ -92,10 +92,6 @@ class _AccountTile extends ConsumerWidget {
|
||||
PopupMenuButton<_AccountAction>(
|
||||
onSelected: (action) => _onAction(context, action),
|
||||
itemBuilder: (_) => const [
|
||||
PopupMenuItem(
|
||||
value: _AccountAction.allMailboxes,
|
||||
child: Text('All mailboxes'),
|
||||
),
|
||||
PopupMenuItem(
|
||||
value: _AccountAction.syncLog,
|
||||
child: Text('Sync log'),
|
||||
@@ -113,16 +109,12 @@ class _AccountTile extends ConsumerWidget {
|
||||
),
|
||||
],
|
||||
),
|
||||
onTap: () => context.push(
|
||||
'/accounts/${account.id}/mailboxes/${Uri.encodeComponent('INBOX')}/emails',
|
||||
),
|
||||
onTap: () => context.push('/accounts/${account.id}/mailboxes'),
|
||||
);
|
||||
}
|
||||
|
||||
Future<void> _onAction(BuildContext context, _AccountAction action) async {
|
||||
switch (action) {
|
||||
case _AccountAction.allMailboxes:
|
||||
await context.push('/accounts/${account.id}/mailboxes');
|
||||
case _AccountAction.syncLog:
|
||||
await context.push('/accounts/${account.id}/sync-log');
|
||||
case _AccountAction.edit:
|
||||
@@ -156,4 +148,4 @@ class _AccountTile extends ConsumerWidget {
|
||||
}
|
||||
}
|
||||
|
||||
enum _AccountAction { allMailboxes, syncLog, edit, delete }
|
||||
enum _AccountAction { syncLog, edit, delete }
|
||||
|
||||
Reference in New Issue
Block a user