Implement filterable issue list panel in banner
epic-bufdir-report-preview-ui-components-task-008 — Extend BufdirValidationSummaryBanner with an expandable issue list panel that slides down when the banner is tapped. The panel lists all BufdirValidationIssue items with filter toggles for errors and warnings. Each issue row displays the section title and issue message. Tapping an issue item calls the focus management utility to scroll to and highlight the corresponding BufdirFieldRowWidget.
Acceptance Criteria
Technical Requirements
Execution Context
Tier 3 - 413 tasks
Can start after Tier 2 completes
Implementation Notes
Extract BufdirIssueListPanel as a separate StatefulWidget to isolate filter state from the banner's animation state — avoids unnecessary parent rebuilds on filter toggle. Use a Riverpod StateProvider
Avoid nested Scrollables without physics: NeverScrollableScrollPhysics on the inner list if the panel height is unconstrained.
Testing Requirements
Widget tests: (1) panel is not visible on initial render; (2) tapping banner header shows panel; (3) tapping again hides panel; (4) with filter 'Errors' only active, warning-severity rows are not present in widget tree; (5) with filter 'Warnings' only active, error-severity rows are not present; (6) tapping an issue row calls a mock BufdirFocusManager.scrollToField with correct fieldIdentifier; (7) panel list is scrollable when issue count exceeds visible height; (8) golden test for expanded panel with mixed issues. Use a mock/fake BufdirFocusManager injected via Riverpod override.
Implementing the tap-to-scroll-and-focus behavior from the validation banner to a specific field row in a long scrollable list is complex in Flutter. If focus management is incorrectly implemented, VoiceOver users who navigate to the banner and select an issue will not be moved to the relevant field row, breaking the accessibility workflow and violating WCAG 2.4.3 (Focus Order).
Mitigation & Contingency
Mitigation: Use BufdirAccessibilityUtils focus management utilities (built in the foundation epic) with explicit GlobalKey-based scroll anchors on each field row. Test with a real iOS device running VoiceOver during widget development, not only in the Flutter accessibility inspector.
Contingency: If programmatic scroll-to-focus cannot be reliably achieved before the TestFlight deadline, fall back to a navigation approach where tapping a banner issue opens a modal detail sheet for that field row rather than scrolling in place, and file a follow-up ticket for the inline scroll implementation.
The validation summary banner must reactively update its issue count as underlying aggregated data changes (e.g., if the coordinator has navigated away and data was refreshed). If the banner's Riverpod provider is not correctly scoped, it may display stale issue counts or fail to disappear when all issues are resolved, eroding coordinator trust in the validation system.
Mitigation & Contingency
Mitigation: Drive the banner exclusively from the same Riverpod provider that powers the full preview model — do not maintain a separate local state for issue counts. Write a widget test that simulates a data refresh mid-review and asserts the banner updates within one frame.
Contingency: If stale state reaches production, add a manual refresh button to the banner as a short-term workaround while the provider scoping is corrected in the next release cycle.