A couple minor notes.
http://gwt-code-reviews.appspot.com/51830/diff/2003/2004 File user/javadoc/com/google/gwt/examples/LayoutPanelExample.java (right): http://gwt-code-reviews.appspot.com/51830/diff/2003/2004#newcode32 Line 32: Widget childOne = new HTML(), childTwo = new HTML(); You should add some text to these HTML widgets so that if somebody uses the example they will actually see childOne and childTwo. http://gwt-code-reviews.appspot.com/51830/diff/2003/2010 File user/src/com/google/gwt/layout/client/Layout.java (right): http://gwt-code-reviews.appspot.com/51830/diff/2003/2010#newcode302 Line 302: public void animate(int duration) { Didn't we agree that this would become layout(int duration)? I see that we now have LayoutPanel#layout(int), but why not here? http://gwt-code-reviews.appspot.com/51830/diff/2003/2010#newcode313 Line 313: public void animate(int duration, final AnimationCallback callback) { And this would be layout(int duration, final AnimationCallback callback)? http://gwt-code-reviews.appspot.com/51830/diff/2003/2014 File user/src/com/google/gwt/user/client/ui/ProvidesLayout.java (right): http://gwt-code-reviews.appspot.com/51830/diff/2003/2014#newcode29 Line 29: * public void onLayout() { Can we move this to a static helper method in this interface to avoid code duplication? It could take an Iterator<?> and look for instances of RequiresLayout. http://gwt-code-reviews.appspot.com/51830/diff/2003/2006 File user/test/com/google/gwt/layout/client/LayoutTest.java (right): http://gwt-code-reviews.appspot.com/51830/diff/2003/2006#newcode57 Line 57: return "com.google.gwt.layout.Layout"; Should be com.google.gwt.layout.LayoutTest http://gwt-code-reviews.appspot.com/51830 --~--~---------~--~----~------------~-------~--~----~ http://groups.google.com/group/Google-Web-Toolkit-Contributors -~----------~----~----~----~------~----~------~--~---
