You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by hl...@apache.org on 2008/10/13 21:02:26 UTC
svn commit: r704205 -
/tapestry/tapestry5/trunk/tapestry-core/src/main/resources/org/apache/tapestry5/corelib/components/palette.js
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();