artificery/dew
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(tui): keep highlight on moved ticket in destination column

Browse source 
After < or > move, find the ticket's index in the destination column's
filtered list via indexWhere instead of jumping to nt.length - 1.
indexWhere returns -1 if not found (e.g. filtered out), max(0,...) clamps
that safely to the first position.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This commit is contained in:
Chris Hendrickson 2026-04-25 13:40:28 -04:00
parent 7f7dd25d76
commit 7bc7ee9898
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
packages/kanban/lib/src/commands/tui_command.dart
View file

@ -417,7 +417,7 @@ class TuiCommand extends DewCommand {
byColumn = _groupByColumn(tickets, config); byColumn = _groupByColumn(tickets, config);
colIdx--; colIdx--;
final nt = _filtered(byColumn[newColId] ?? [], searchQuery); final nt = _filtered(byColumn[newColId] ?? [], searchQuery);
ticketIdx = max(0, nt.length - 1); ticketIdx = max(0, nt.indexWhere((x) => x.id == t.id));
statusMsg = 'Moved ${t.id} → ${config.columns[colIdx].name}'; statusMsg = 'Moved ${t.id} → ${config.columns[colIdx].name}';
} on ArgumentError catch (e) { } on ArgumentError catch (e) {
statusMsg = 'Error: ${e.message ?? e}'; statusMsg = 'Error: ${e.message ?? e}';
@ -433,7 +433,7 @@ class TuiCommand extends DewCommand {
byColumn = _groupByColumn(tickets, config); byColumn = _groupByColumn(tickets, config);
colIdx++; colIdx++;
final nt = _filtered(byColumn[newColId] ?? [], searchQuery); final nt = _filtered(byColumn[newColId] ?? [], searchQuery);
ticketIdx = max(0, nt.length - 1); ticketIdx = max(0, nt.indexWhere((x) => x.id == t.id));
statusMsg = 'Moved ${t.id} → ${config.columns[colIdx].name}'; statusMsg = 'Moved ${t.id} → ${config.columns[colIdx].name}';
} on ArgumentError catch (e) { } on ArgumentError catch (e) {
statusMsg = 'Error: ${e.message ?? e}'; statusMsg = 'Error: ${e.message ?? e}';