tapestry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From hls...@apache.org
Subject svn commit: r704205 - /tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/palette.js
Date Mon, 13 Oct 2008 19:02:26 GMT
Author: hlship
Date: Mon Oct 13 12:02:26 2008
New Revision: 704205

URL: http://svn.apache.org/viewvc?rev=704205&view=rev
Log:
TAP5-261: Palette option reordering broken in IE

Modified:
    tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/palette.js

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/palette.js
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/palette.js?rev=704205&r1=704204&r2=704205&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/palette.js
(original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/palette.js
Mon Oct 13 12:02:26 2008
@@ -214,7 +214,11 @@
             }.bind(this));
             if (candidate) before = candidate;
         }
+        this.addOption(to, option, before);
+    },
 
+    addOption : function(to, option, before)
+    {
         try
         {
             to.add(option, before);
@@ -233,6 +237,7 @@
                 to.add(option, before.index);
             }
         }
+
     },
 
     moveDownClicked : function(event)
@@ -254,7 +259,7 @@
     {
         movers.each(function(option)
         {
-            this.selected.add(option, before);
+            this.addOption(this.selected, option, before);
         }.bind(this));
 
         this.updateHidden();



Mime
View raw message