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

Unified Diff: sky/sdk/lib/framework/components2/popup_menu.dart

Issue 1174253003: Make the stocks popup menu fade in (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 6 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « sky/engine/platform/graphics/GraphicsContext.h.rej ('k') | sky/sdk/lib/framework/fn2.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/sdk/lib/framework/components2/popup_menu.dart
diff --git a/sky/sdk/lib/framework/components2/popup_menu.dart b/sky/sdk/lib/framework/components2/popup_menu.dart
index d264f575f19d2dcbaddeee5c1d607306c3382cfe..d41f03a82f0353b6f0f73927c6495847554694a0 100644
--- a/sky/sdk/lib/framework/components2/popup_menu.dart
+++ b/sky/sdk/lib/framework/components2/popup_menu.dart
@@ -114,14 +114,17 @@ class PopupMenu extends AnimatedComponent {
}));
// inlineStyle: _inlineStyle(),
- return new ShrinkWrapWidth(
- child: new Container(
- padding: const EdgeDims.all(8.0),
- decoration: new BoxDecoration(
- backgroundColor: Grey[50],
- borderRadius: 2.0,
- boxShadow: Shadow[level]),
- child: new Block(children)
+ return new Opacity(
+ opacity: math.min(1.0, _position * 3.0),
+ child: new ShrinkWrapWidth(
+ child: new Container(
+ padding: const EdgeDims.all(8.0),
+ decoration: new BoxDecoration(
+ backgroundColor: Grey[50],
+ borderRadius: 2.0,
+ boxShadow: Shadow[level]),
+ child: new Block(children)
+ )
)
);
}
« no previous file with comments | « sky/engine/platform/graphics/GraphicsContext.h.rej ('k') | sky/sdk/lib/framework/fn2.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698