- 09 6月, 2015 7 次提交
-
-
由 Adam Barth 提交于
The popup menu in the stocks app is supposed to size its width to the max intrinsic width of the menu. This CL teaches it how to do that. It's a shame that we need to ceilToDouble the output of RenderParagraph. If we don't do that, we run into floating point layout trouble and the menu triggers a line break. The correct fix is to do layout in fixed point. R=ianh@google.com Review URL: https://codereview.chromium.org/1168113005
-
由 Eric Seidel 提交于
I don't think this is the final API we're going to use. I suspect we'll add a TextStyle struct to hold color and size, etc. which back-ends into CSS like this does today. Currently no one uses this color even though it exists. This is one step towards fixing: https://github.com/domokit/mojo/issues/213 R=ianh@google.com Review URL: https://codereview.chromium.org/1162573007
-
由 Adam Barth 提交于
This CL splits getIntrinsicDimensions into getMinIntrinsicWidth, getMaxIntrinsicWidth, getMinIntrinsicHeight, and getMaxIntrinsicHeight so that we can properly shrink-wrap the stocks app menu. This CL just contains the refactoring. The use in stocks app will come in a later CL. R=ianh@google.com Review URL: https://codereview.chromium.org/1167293003
-
由 Collin Jackson 提交于
R=abarth@chromium.org, abarth Review URL: https://codereview.chromium.org/1164363002
-
由 Adam Barth 提交于
R=eseidel@chromium.org, ianh@google.com Review URL: https://codereview.chromium.org/1166183002
-
由 Adam Barth 提交于
This CL is the first step towards getting PopupMenuItem working in fn2. We introduce the ability to add a minWidth to a Container by creating a RenderConstrainedBox class to apply the new constraints. R=ianh@google.com Review URL: https://codereview.chromium.org/1155683011
-
由 Adam Barth 提交于
And deploy it in a few places. R=ianh@google.com Review URL: https://codereview.chromium.org/1164303002
-
- 06 6月, 2015 5 次提交
-
-
由 Eric Seidel 提交于
R=abarth@chromium.org Review URL: https://codereview.chromium.org/1156383006
-
由 Adam Barth 提交于
We were hitting an assert in RenderCustomPaint that the callback wasn't null. R=eseidel@chromium.org, jackson@chromium.org Review URL: https://codereview.chromium.org/1162023010
-
由 Hixie 提交于
Export Point, Size, Rect, Color, Paint, Path, BoxDecoration, Border, BorderSide, EdgeDims, and FlexDirection from fn2.dart That way most of our libraries don't have to use sky.Foo all over the place, and we can reduce the number of imports in most Dart components. R=abarth@chromium.org Review URL: https://codereview.chromium.org/1161323004
-
由 Adam Barth 提交于
The + isn't quite centered properly, but the button is a circle in the right place. R=eseidel@chromium.org, ianh@google.com Review URL: https://codereview.chromium.org/1156383004
-
由 Hixie 提交于
- several places where we use a getter knowing that it will return a subclass of its declared type, even asserting that it does, but where the analyzer is worried that the getter might start returning a new value unexpectedly, solved by having a temporary local variable shadow the getter and asserting that it doesn't change value - many many places where we do this with parentData specifically, solved by type-erasing parentData. - a place where a mixin wants to be subclassing another class, and uses its methods, solved by saying that the mixin is abstract but implements the superclass R=abarth@chromium.org Review URL: https://codereview.chromium.org/1160923008
-
- 05 6月, 2015 17 次提交
-
-
由 Collin Jackson 提交于
R=abarth@chromium.org, eseidel@chromium.org, abarth Review URL: https://codereview.chromium.org/1157573012
-
由 Hixie 提交于
Rename "BorderSide.None" to "BorderSide.none", since it's a constant, and change the sky.Size.infinite() constructor to a constant. R=abarth@chromium.org Review URL: https://codereview.chromium.org/1158263005
-
由 Hixie 提交于
Include DETACHED, NEEDS-LAYOUT, and relayoutSubtreeRoot info in the debug output for the render tree R=chinmaygarde@google.com Review URL: https://codereview.chromium.org/1149083011
-
由 Hixie 提交于
Improve the RenderObject.toString() logic so that there's not blank lines everywhere and to generally be better. Adds sizes, positions, transforms, improves RenderParagraph's output. R=abarth@chromium.org Review URL: https://codereview.chromium.org/1160233004
-
由 Eric Seidel 提交于
The goal of this is to have a Renderer-layer abstraction around the Sky Engine c++ apis for drawing a shadow. R=mpcomplete@chromium.org Review URL: https://codereview.chromium.org/1146893004
-
由 Hixie 提交于
R=abarth@chromium.org Review URL: https://codereview.chromium.org/1151533006
-
由 Hixie 提交于
R=eseidel@chromium.org Review URL: https://codereview.chromium.org/1162623011
-
由 Hixie 提交于
This makes debugging the render tree a lot easier. Just print the node you care about, and you get an indented tree view of its subtree, including settings. New subclasses should implement the new virtual method debugDescribeSettings() to expose new settings. R=eseidel@chromium.org Review URL: https://codereview.chromium.org/1157993005
-
由 Adam Barth 提交于
This CL makes FixedHeightScrollable able to measure heights. In the DOM-based version, FixedHeightScrollable forced a synchronous layout in a microtask in order to measure the size of itself and its children. In the DOM-less version, we now require the subclass to tell us the item height and we measure our own height using a layout callback. R=ianh@google.com Review URL: https://codereview.chromium.org/1161063005
-
由 Hixie 提交于
https://www.dartlang.org/articles/style-guide/#names R=jackson@google.com Review URL: https://codereview.chromium.org/1158813004
-
由 Matt Perry 提交于
R=abarth@chromium.org Review URL: https://codereview.chromium.org/1169583002
-
由 Hixie 提交于
R=abarth@chromium.org Review URL: https://codereview.chromium.org/1144933007
-
由 Adam Barth 提交于
In the future, we might want to extend this class to clip to a configurable rect in addition to clipping to its bounds. R=eseidel@chromium.org, ianh@google.com Review URL: https://codereview.chromium.org/1145263006
-
由 Collin Jackson 提交于
R=abarth@chromium.org, ianh@google.com, abarth, hixie Review URL: https://codereview.chromium.org/1165983002
-
由 Hixie 提交于
Currently supports only solid borders, but of any colour and size, independently controlling each side. R=abarth@chromium.org Review URL: https://codereview.chromium.org/1154903006
-
由 Hixie 提交于
R=abarth@chromium.org Review URL: https://codereview.chromium.org/1165013003
-
由 Hixie 提交于
The root cause of the crash was threefold: - you can't enumerate enums, in dart; you have to enumerate their values. - RenderNodeWrapper was trying to call remove(child.root) on its root, but that's not something that all RenderNodes have - when a RenderNode was deparented, its relayoutSubtreeRoot wasn't removed, so the node ended up in an inconsistent state regarding its parent tree R=abarth@chromium.org Review URL: https://codereview.chromium.org/1151573020
-
- 04 6月, 2015 9 次提交
-
-
由 Hixie 提交于
R=jackson@google.com Review URL: https://codereview.chromium.org/1166523006
-
由 Adam Barth 提交于
This CL makes gesture events work again in fn2. R=ianh@google.com Review URL: https://codereview.chromium.org/1146923005
-
由 Collin Jackson 提交于
R=abarth@chromium.org, abarth Review URL: https://codereview.chromium.org/1168623002
-
由 Collin Jackson 提交于
This also fixes a bug that was preventing markNeedsLayout from working R=abarth@chromium.org, eseidel@chromium.org, abarth, eseidel Review URL: https://codereview.chromium.org/1160013004
-
由 Adam Barth 提交于
This CL introduces RenderStack and fixes a number of bugs to make the Drawer in components2 work. R=ianh@google.com, eseidel@chromium.org Review URL: https://codereview.chromium.org/1147143005
-
由 Matt Perry 提交于
Also add operator==, hashCode, toString, and some basic Color constants to Color. R=abarth@chromium.org, ianh@google.com Review URL: https://codereview.chromium.org/1162023004
-
由 Eric Seidel 提交于
R=ianh@google.com Review URL: https://codereview.chromium.org/1153413003
-
由 Hixie 提交于
Previously, we were not adjusting the minimum width, so we ended up offsetting the child but not shrinking it, when the parent expected the child to be exactly fit to its dimensions. R=abarth@chromium.org Review URL: https://codereview.chromium.org/1155303005
-
由 Adam Barth 提交于
These transforms are currently very basic but they seem to mostly work. R=eseidel@chromium.org Review URL: https://codereview.chromium.org/1152273004
-
- 03 6月, 2015 2 次提交
-
-
由 Eric Seidel 提交于
Looks horrible. But it's a start. R=abarth@chromium.org, ianh@google.com BUG= Review URL: https://codereview.chromium.org/1146913005
-
由 Adam Barth 提交于
This CL is a first pass at implementing container in fn2. In this approach, Container is a Component that builds a number of RenderNodeWrappers depending on what is needed. R=ianh@google.com Review URL: https://codereview.chromium.org/1158983005
-