Skip to content

Commit 6dcd440

Browse files
chrisbobbegnprice
authored andcommitted
compose [nfc]: Refactor _Banner label's vertical padding
In the Figma: https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=4010-6425&m=dev https://www.figma.com/design/1JTNtYo9memgW7vV6d0ygq/Zulip-Mobile?node-id=4031-17028&m=dev the banner label's 9px vertical padding is factored as 5px applied to the whole banner, and 4px applied to just the label, excluding trailing buttons. Follow that, so it can work as intended when we add such buttons.
1 parent 0ebe1b9 commit 6dcd440

File tree

1 file changed

+13
-11
lines changed

1 file changed

+13
-11
lines changed

lib/widgets/compose_box.dart

+13-11
Original file line numberDiff line numberDiff line change
@@ -1400,17 +1400,19 @@ abstract class _Banner extends StatelessWidget {
14001400
minimum: const EdgeInsetsDirectional.only(start: 8)
14011401
// (SafeArea.minimum doesn't take an EdgeInsetsDirectional)
14021402
.resolve(Directionality.of(context)),
1403-
child: Row(
1404-
children: [
1405-
Expanded(
1406-
child: Padding(
1407-
padding: const EdgeInsetsDirectional.fromSTEB(8, 9, 0, 9),
1408-
child: Text(style: labelTextStyle,
1409-
label))),
1410-
const SizedBox(width: 8),
1411-
// TODO(#720) "x" button for the error banner goes here.
1412-
// 24px square with 8px touchable padding in all directions?
1413-
])));
1403+
child: Padding(
1404+
padding: const EdgeInsetsDirectional.fromSTEB(8, 5, 0, 5),
1405+
child: Row(
1406+
children: [
1407+
Expanded(
1408+
child: Padding(
1409+
padding: const EdgeInsets.symmetric(vertical: 4),
1410+
child: Text(style: labelTextStyle,
1411+
label))),
1412+
const SizedBox(width: 8),
1413+
// TODO(#720) "x" button for the error banner goes here.
1414+
// 24px square with 8px touchable padding in all directions?
1415+
]))));
14141416
}
14151417
}
14161418

0 commit comments

Comments
 (0)