diff --git a/lib/pantallas/pantalla_alarma_sonando.dart b/lib/pantallas/pantalla_alarma_sonando.dart index cf62ff1..0ce1d67 100644 --- a/lib/pantallas/pantalla_alarma_sonando.dart +++ b/lib/pantallas/pantalla_alarma_sonando.dart @@ -339,14 +339,18 @@ class _PantallaAlarmaSonandoState extends State { ], ), ), - const SizedBox(height: 10), + // Issue 3 (feedback-pruebas): t4:427 wraps POSPONER's + // eyebrow, the snooze tiles and Stop in a `gap:12` flex + // column -- the same 12 on both sides, not the 10/14 pair + // this used to carry. + const SizedBox(height: 12), _FilaSnoozeFija( alarma: alarma, l10n: l10n, tokens: tokens, onPosponer: _posponer, ), - const SizedBox(height: 14), + const SizedBox(height: 12), SizedBox( width: double.infinity, child: FilledButton.icon( diff --git a/test/pantallas/pantalla_alarma_sonando_tier5_test.dart b/test/pantallas/pantalla_alarma_sonando_tier5_test.dart index b241773..f3277ab 100644 --- a/test/pantallas/pantalla_alarma_sonando_tier5_test.dart +++ b/test/pantallas/pantalla_alarma_sonando_tier5_test.dart @@ -159,4 +159,49 @@ void main() { await tester.pumpAndSettle(); }, ); + + testWidgets( + 'Issue 3 (feedback-pruebas): the gap above the snooze tiles matches the ' + 'gap below them (t4:427 draws a uniform gap:12 flex column) -- the ' + 'previous 10/14 pair matched neither the prototype nor each other', + (tester) async { + await _montarPantalla(tester); + final l10n = AppLocalizations.of( + tester.element(find.byType(PantallaAlarmaSonando)), + ); + + final eyebrowBottom = + tester + .getBottomLeft( + find + .ancestor( + of: find.byIcon(Icons.snooze_rounded), + matching: find.byType(Row), + ) + .first, + ) + .dy; + final tileDestacado = find.ancestor( + of: find.text(l10n.alarmSnoozeOptionLabel(10)), + matching: find.byType(FilledButton), + ); + final tilesTop = tester.getTopLeft(tileDestacado).dy; + final tilesBottom = tester.getBottomLeft(tileDestacado).dy; + final stopButtonTop = + tester + .getTopLeft(find.byKey(const ValueKey('ringing-stop-button'))) + .dy; + + expect( + tilesTop - eyebrowBottom, + 12, + reason: 't4:427: gap:12 above the snooze tiles', + ); + expect( + stopButtonTop - tilesBottom, + 12, + reason: 't4:427: gap:12 below the snooze tiles, same as above', + ); + }, + ); }