Reviewers: Jasvir, metaweta,

Description:
1. build.xml has a rule to check if the playground needs to be rebuilt.
The rule doesn't look at all relevant files.

2. tweak button layout.

Please review this at http://codereview.appspot.com/6180045/

Affected files:
  M     build.xml
  M     src/com/google/caja/demos/playground/client/ui/PlaygroundUI.ui.xml


Index: src/com/google/caja/demos/playground/client/ui/PlaygroundUI.ui.xml
===================================================================
--- src/com/google/caja/demos/playground/client/ui/PlaygroundUI.ui.xml (revision 4867) +++ src/com/google/caja/demos/playground/client/ui/PlaygroundUI.ui.xml (working copy)
@@ -66,20 +66,20 @@
             </g:center>
             <g:north size="30">
<g:DockLayoutPanel styleName="{style.playgroundUI}" unit='PX'>
+                <g:west size="50">
+                  <g:Button ui:field="cajoleButton">
+                    Cajole
+                  </g:Button>
+                </g:west>
+                <g:west size="50">
+                  <g:Button ui:field="goButton">
+                    Load
+                  </g:Button>
+                </g:west>
                 <g:center>
                   <g:SuggestBox ui:field="addressField" width="100%">
                   </g:SuggestBox>
                 </g:center>
-                <g:east size="200">
-                  <g:HorizontalPanel>
-                    <g:Button width="100" ui:field="goButton">
-                      Load
-                    </g:Button>
-                    <g:Button width="100" ui:field="cajoleButton">
-                      Cajole
-                    </g:Button>
-                  </g:HorizontalPanel>
-                </g:east>
               </g:DockLayoutPanel>
             </g:north>
           </g:DockLayoutPanel>
@@ -93,23 +93,25 @@
               </g:center>
               <g:north size="30">
<g:DockLayoutPanel styleName="{style.playgroundUI}" unit='PX'>
+                  <g:west size="50">
+                    <g:Button ui:field="clearButton">
+                      Clear
+                    </g:Button>
+                  </g:west>
+                  <g:west size="70">
+                    <g:Button ui:field="defaultButton">
+                      Default
+                    </g:Button>
+                  </g:west>
+                  <g:west size="50">
+                    <g:Button ui:field="loadButton">
+                      Load
+                    </g:Button>
+                  </g:west>
                   <g:center>
<g:SuggestBox ui:field="policyAddressField" width="100%">
                     </g:SuggestBox>
                   </g:center>
-                  <g:east size="250">
-                    <g:HorizontalPanel>
-                      <g:Button ui:field="loadButton">
-                        Load
-                      </g:Button>
-                      <g:Button ui:field="clearButton">
-                        Clear
-                      </g:Button>
-                      <g:Button ui:field="defaultButton">
-                        Default
-                      </g:Button>
-                    </g:HorizontalPanel>
-                  </g:east>
                 </g:DockLayoutPanel>
               </g:north>
             </g:DockLayoutPanel>
Index: build.xml
===================================================================
--- build.xml   (revision 4867)
+++ build.xml   (working copy)
@@ -1225,8 +1225,7 @@
     <uptodate property="uptodate.playground"
         targetfile="${war}/playground/playground.nocache.js">
<srcfiles dir="${src.caja}/demos/playground/client" includes="**/*.java"/>
-      <srcfiles dir="${src.caja}/demos/playground"
-          includes="Playground*.gwt.xml"/>
+      <srcfiles dir="${src.caja}/demos/playground" includes="**/*.xml"/>
     </uptodate>
   </target>



Reply via email to