Embed PeriodComparisonWidget in all summary views
epic-periodic-summaries-ui-task-010 — Integrate the reusable PeriodComparisonWidget into both the PeriodicSummaryCard and the coordinator team mentor detail view. Ensure the widget correctly receives prior and current period data from the parent BLoC. Validate that offline-cached data renders correctly in the widget when no network is available. Confirm layout does not overflow on smaller screen sizes.
Acceptance Criteria
Technical Requirements
Execution Context
Tier 5 - 253 tasks
Can start after Tier 4 completes
Implementation Notes
PeriodComparisonWidget should be a pure stateless widget with no BLoC dependency — it receives PeriodSummaryData via constructor. This makes it reusable across any parent. Guard against division by zero in delta calculations: if priorPeriod.hours == 0 and currentPeriod.hours > 0, show '+∞%' or 'New' rather than crashing. If both are zero, show '—'.
For layout safety on 320px screens, use Flexible(child: Text(..., overflow: TextOverflow.ellipsis)) inside Row widgets rather than fixed-width containers. The offline indicator should be driven by an optional isOffline: bool parameter defaulting to false — this keeps the widget decoupled from connectivity detection logic.
Testing Requirements
Widget tests: render with valid currentPeriod + priorPeriod (verify delta indicators); render with priorPeriod = null (verify no crash, placeholder shown); render with all-zero priorPeriod (verify delta is 0% or 'N/A', not NaN/infinity); render with offline cached data flag (verify offline indicator visible); verify no overflow at 320px width using tester.binding.window.physicalSizeTestValue. Golden tests: 320px and 390px widths, light and dark themes, for both the PeriodicSummaryCard embed and the mentor detail embed.
If the cached summary is from a prior period (e.g., previous quarter's card is still cached), the digest card could appear on the home screen with outdated numbers after a new period begins, confusing users who have already seen that data.
Mitigation & Contingency
Mitigation: In the SummaryCacheRepository, store the period_start and period_end alongside cached data. In the PeriodicSummaryCard BLoC, compare the cached record's period against the current date using PeriodCalculatorService. Only render the card if the cached summary belongs to the active period.
Contingency: If stale cards appear in production, push a hotfix that adds the period validity check to the BLoC and clears all cached summaries older than the current period boundary via a forced cache flush on next app launch.
Using colour alone to distinguish underactive from overloaded mentors in the coordinator view would fail WCAG 1.4.1 (use of colour). This is especially critical for organisations serving users with colour vision deficiency.
Mitigation & Contingency
Mitigation: Always pair colour indicators with a text label or icon (e.g., a downward arrow + 'Underactive' text alongside the amber colour). Use the contrast-safe-color-palette design tokens throughout. Run the contrast-ratio-validator on all new colour combinations in CI.
Contingency: If an accessibility audit flags colour-only indicators post-launch, introduce icon-based indicators as a patch release and update the design token definitions to enforce the paired-indicator pattern going forward.
Inserting a new card into the role-based home screen requires changes to shared home screen widget composition, which may conflict with parallel feature branches also modifying the home screen layout, causing merge conflicts and integration delays.
Mitigation & Contingency
Mitigation: Implement the PeriodicSummaryCard as a fully self-contained widget that the home screen conditionally renders based on a BLoC state flag. Minimise changes to the home screen itself to a single conditional insertion point, reducing the surface area for merge conflicts.
Contingency: If integration conflicts block the PR, isolate the card behind the organisation-scoped feature flag so it can be toggled independently and merged without affecting other home screen changes.