Commit 07d60a7
committed
compose [nfc]: Refactor outer vertical padding on label in _Banner
Move vertical padding so it won't wrap the trailing element, when
present (as a `buildTrailing` that returns non-null). When we add
the "Cancel" and "Save" buttons for the edit-message banner, those
will want to control their own outer vertical padding to make it
respond to taps, because the painted button is supposed to be just
28px tall, which is too small for a touch target; see discussion
linked in dartdoc.
NFC because `buildTrailing` on _Banner's only subclass,
_ErrorBanner, returns null.1 parent d5d0ea8 commit 07d60a7
1 file changed
+9
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1400 | 1400 | | |
1401 | 1401 | | |
1402 | 1402 | | |
1403 | | - | |
| 1403 | + | |
| 1404 | + | |
| 1405 | + | |
| 1406 | + | |
| 1407 | + | |
| 1408 | + | |
| 1409 | + | |
1404 | 1410 | | |
1405 | 1411 | | |
1406 | 1412 | | |
| |||
1431 | 1437 | | |
1432 | 1438 | | |
1433 | 1439 | | |
1434 | | - | |
| 1440 | + | |
1435 | 1441 | | |
1436 | 1442 | | |
1437 | 1443 | | |
1438 | 1444 | | |
1439 | | - | |
| 1445 | + | |
1440 | 1446 | | |
1441 | 1447 | | |
1442 | 1448 | | |
| |||
0 commit comments