Hi Brendan,

> I have ShowHighlight and VariableRowHigh to true. And when mouseOut event
> occurs, I get  IndexOutOfBoundException.
I think that it's something problematic in your data model ... some
week ago I didn't add a small fix to prevent a
IndexOutOfBoundException in case of something strange in table data,
but didn't apply because I think it's better to get the exception and
fix the problem (instead of simply ignoring it, maybe writing
something in log to console) ...

>  I did a bit of trace and found the function getRowAt in TerraTableViewSkin
> incorrectly return 1 when the y ordinate is greater than all rows.
>
> the problem is the with code line in 585, 586 and 587.  After the binary
> search return a -2 when y is bigger than the rowBoundaries
> the code in 587 modified it index to 1.
> In my case, where only one row, so IndexOutOfBoundException.
>
If there aren't objections I think could be safer to force the range
of the returned row in the [0 .. numberOfRows] before the end of the
getRowAt() method ... and maybe write a warning in console logs (but
probably not, unless we'll have a pluggable logging system).
And maybe then the same even in the getColumnAt() method ...
Roger (and all others :-) ) what do you think ?

Anyway, Brendan, if you have a minimal sample that we can run, to
better ensure all will be fixed ... or you can take a look at (under
tests) table_view_test2.bxml and table_view_test3.bxml as a minimal
sample with variableRowHeight:true , and maybe help us reproduce the
issue.

Tell me.
Thank you for now.

Bye,
Sandro

Reply via email to