Hi,

I know this is probably an odd boundary case, but CellTable's
dispatching code doesn't handle coming across another table rendered
within one of its headers. (Unfortunately, yes, we had a sufficiently
complex header that embedding a 1-row table was necessary.)

Nonetheless, someone else had already come across this:

http://code.google.com/p/google-web-toolkit/issues/detail?id=5714

And the patch is pretty simple:

http://code.google.com/p/google-web-toolkit/issues/attachmentText?id=5714&aid=-2665470390260277603&name=fix-celltable-losing-events.diff&token=26ab539a8e6a4558dd5620e169aee9e2

Granted, no tests. It seems like a hard/obscure thing to test for--but I
suppose that's the point of tests.

Can this patch be accepted as is or does it need more work/tests?

Thanks,
Stephen

-- 
http://groups.google.com/group/Google-Web-Toolkit-Contributors

Reply via email to