Re: inmethod DataGrid javascript error (with fix?)

2009-09-25 Thread Matej Knopp
Thanks. Should be fixed in SVN.

-Matej

On Fri, Sep 25, 2009 at 2:05 AM, Russell Morrisey rmorri...@csc.com wrote:
 I'm getting a javascript error using the inmethod DataGrid; it looks to me
 like a bug with the DataGrid's script code. I am hoping that the DataGrid
 script can be patched =)

 When using the inmethod grid (com.inmethod.grid.datagrid.DataGrid), I get
 the error whenever I hover the mouse over a row in the grid. This seems to
 happen:
 -Only in IE6
 -Only when window.update is already defined. In my case, my page has:
 input id=update value=UPDATE type=button /. (This button is just a
 stub, not wicket-enabled yet.) Debugging in Visual Studio, I see that
 there is a reference to the button (update) already defined, which seems
 to be generated automatically by IE.

 When I mouse over the grid I get: Object does not support this property
 or method, in AbstractGrid/res/script.js, in the updatePrelight method,
 on this line:

 update = function(e) {

                                var scrollLeft;

                                if (Wicket.Browser.isOpera()) {
                                        // for some reason opera doesn't
 preserve the scroll offset when changing/removing style
                                        bodyContainer1 = this.getElement(
 div, imxt-body-container1);
                                        scrollLeft =
 bodyContainer1.scrollLeft;
                                        //e.style.visibility = hidden;
                                }

                                if (e.imxtPrelight == true) {
                                        addClass(e, imxt-prelight);
                                } else {
                                        removeClass(e, imxt-prelight);
                                }

                                if (Wicket.Browser.isOpera()) {
                                        //e.style.visibility = ;
                                        bodyContainer1 = this.getElement(
 div, imxt-body-container1);
                                        bodyContainer1.scrollLeft =
 scrollLeft;
                                }

                        }.bind(this);

 We use the DataGrid on another page in our production app and it doesn't
 experience this problem. The other page does not have a DOM element with
 id=update, or a window.update property. I think the script needs to be
 changed to:

 var update = function...

 So that the locally defined function isn't trying to overwrite the
 window.update property.

 I'm using datagrid version: 1.0.0-SNAPSHOT according to our project's POM.

 Thanks!

 Russell E. Morrisey
 Application Designer Associate Professional
 CSC

 NOTE: Regardless of content, this e-mail shall not operate to bind CSC to
 any order or other contract unless pursuant to explicit written agreement
 or government initiative expressly permitting the use of e-mail for such
 purpose.

-
To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org
For additional commands, e-mail: users-h...@wicket.apache.org



inmethod DataGrid javascript error (with fix?)

2009-09-24 Thread Russell Morrisey
I'm getting a javascript error using the inmethod DataGrid; it looks to me 
like a bug with the DataGrid's script code. I am hoping that the DataGrid 
script can be patched =)

When using the inmethod grid (com.inmethod.grid.datagrid.DataGrid), I get 
the error whenever I hover the mouse over a row in the grid. This seems to 
happen:
-Only in IE6
-Only when window.update is already defined. In my case, my page has: 
input id=update value=UPDATE type=button /. (This button is just a 
stub, not wicket-enabled yet.) Debugging in Visual Studio, I see that 
there is a reference to the button (update) already defined, which seems 
to be generated automatically by IE.

When I mouse over the grid I get: Object does not support this property 
or method, in AbstractGrid/res/script.js, in the updatePrelight method, 
on this line:

update = function(e) {
 
var scrollLeft;
 
if (Wicket.Browser.isOpera()) {
// for some reason opera doesn't 
preserve the scroll offset when changing/removing style
bodyContainer1 = this.getElement(
div, imxt-body-container1); 
scrollLeft = 
bodyContainer1.scrollLeft;
//e.style.visibility = hidden;
} 
 
if (e.imxtPrelight == true) {
addClass(e, imxt-prelight);
} else {
removeClass(e, imxt-prelight);
}
 
if (Wicket.Browser.isOpera()) {
//e.style.visibility = ;
bodyContainer1 = this.getElement(
div, imxt-body-container1);
bodyContainer1.scrollLeft = 
scrollLeft;
}
 
}.bind(this);

We use the DataGrid on another page in our production app and it doesn't 
experience this problem. The other page does not have a DOM element with 
id=update, or a window.update property. I think the script needs to be 
changed to:

var update = function... 

So that the locally defined function isn't trying to overwrite the 
window.update property.

I'm using datagrid version: 1.0.0-SNAPSHOT according to our project's POM.

Thanks! 

Russell E. Morrisey
Application Designer Associate Professional
CSC 

NOTE: Regardless of content, this e-mail shall not operate to bind CSC to 
any order or other contract unless pursuant to explicit written agreement 
or government initiative expressly permitting the use of e-mail for such 
purpose.