Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(28)

Side by Side Diff: sky/sdk/lib/widgets/popup_menu_item.dart

Issue 1215613005: Improve the padding and sizing of popup menus. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 import '../painting/text_style.dart'; 5 import '../painting/text_style.dart';
6 import 'basic.dart'; 6 import 'basic.dart';
7 import 'default_text_style.dart'; 7 import 'default_text_style.dart';
8 import 'ink_well.dart'; 8 import 'ink_well.dart';
9 import 'theme.dart'; 9 import 'theme.dart';
10 10
11 class PopupMenuItem extends Component { 11 class PopupMenuItem extends Component {
12 PopupMenuItem({ String key, this.child, this.opacity}) : super(key: key); 12 PopupMenuItem({ String key, this.child, this.opacity}) : super(key: key);
13 13
14 final Widget child; 14 final Widget child;
15 final double opacity; 15 final double opacity;
16 16
17 TextStyle get textStyle => Theme.of(this).text.subhead; 17 TextStyle get textStyle => Theme.of(this).text.subhead;
18 18
19 Widget build() { 19 Widget build() {
20 return new Opacity( 20 return new Opacity(
21 opacity: opacity, 21 opacity: opacity,
22 child: new InkWell( 22 child: new InkWell(
23 child: new Container( 23 child: new Container(
24 constraints: const BoxConstraints(minWidth: 112.0), 24 height: 48.0,
25 padding: const EdgeDims.all(16.0),
26 child: new DefaultTextStyle( 25 child: new DefaultTextStyle(
27 style: textStyle, 26 style: textStyle,
28 child: child 27 child: child
29 ) 28 )
30 ) 29 )
31 ) 30 )
32 ); 31 );
33 } 32 }
34 } 33 }
OLDNEW
« sky/sdk/lib/rendering/box.dart ('K') | « sky/sdk/lib/widgets/popup_menu.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698