committed as r10218 before I read Ray's comment. On 2011/05/24 20:06:24, rjrjr wrote:
Rather than detaching, could you render the new contents in a hidden
div and
then replace the tbody?
I suppose that would work too, but the bug is only in Firefox 3.6 and earlier. I'm not sure it would make much of a difference because we still end up with a detach, and attach, and an innerHTML call.
IE is a little tricky about doing this kind of thing with table
elements,
but HTMLPanel.HTMLPanel(String, String) shows the workaround for that
(just
render inside a div and all is forgiven).
That's an understatement. You cannot remove a tbody in IE, nor can you set its innerHTML. Instead, we create a new table in a detached div, as you suggest, then move every row from the new tbody into the existing tbody.
On Tue, May 24, 2011 at 12:50 PM, <mailto:[email protected]> wrote:
> In Firefox 3.6 and older, detaching a tbody from a table causes
Firefox
> to convert all of the TD elements to divs. Firefox 4.0 doesn't have > this problem and works as expected. > > I updated the patch to skip the detach in Firefox 3.6-. > > > http://gwt-code-reviews.appspot.com/1443805/ > > -- > http://groups.google.com/group/Google-Web-Toolkit-Contributors >
http://gwt-code-reviews.appspot.com/1443805/ -- http://groups.google.com/group/Google-Web-Toolkit-Contributors
