From 492f5919c7e89afcd73e52df35a2b6c4451a1c09 Mon Sep 17 00:00:00 2001 From: Adam Barth Date: Wed, 10 Jun 2015 14:58:21 -0700 Subject: [PATCH] Rows in stocks app should have ink splashes R=ianh@google.com Review URL: https://codereview.chromium.org/1174153002. --- examples/stocks2/lib/stock_row.dart | 21 +++++++++++---------- 1 file changed, 11 insertions(+), 10 deletions(-) diff --git a/examples/stocks2/lib/stock_row.dart b/examples/stocks2/lib/stock_row.dart index 6df131c520f..46563dda93e 100644 --- a/examples/stocks2/lib/stock_row.dart +++ b/examples/stocks2/lib/stock_row.dart @@ -36,15 +36,16 @@ class StockRow extends Component { new FlexExpandingChild(new Text(changeInPrice), key: "changeInPrice") ]; - return new Container( - padding: const EdgeDims(16.0, 16.0, 20.0, 16.0), - height: kHeight, // TODO(hansmuller): This shouldn't be needed - decoration: const BoxDecoration( - backgroundColor: const sky.Color(0xFFFFFFFF), - border: const Border( - bottom: const BorderSide( - color: const sky.Color(0xFFF4F4F4), - width: 1.0))), - child: new FlexContainer(children: children)); + // TODO(hansmuller): An explicit |height| shouldn't be needed + return new InkWell(children: [ + new Container( + padding: const EdgeDims(16.0, 16.0, 20.0, 16.0), + height: kHeight, + decoration: const BoxDecoration( + border: const Border( + bottom: const BorderSide(color: const sky.Color(0xFFF4F4F4)))), + child: new FlexContainer(children: children) + ) + ]); } }