Bug 1572777 Comment 3 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

(In reply to Nicolas Chevobbe [:nchevobbe] from comment #2)
> Yeah, I tried reverting back to a plain table with sticky th at some point, but that wasn't working well.

I think my patch is working pretty well, but there may be a slight difference in the borders. That might also be fixable though.

It's perhaps worth noting that Chrome doesn't have fixed headers - maybe it's not needed?

> No idea how the grid gets translated into a string when copying (this has always been a very consuming thing to get right)
Maybe an alternative solution is to change the logic here https://dxr.mozilla.org/mozilla-central/source/dom/base/nsPlainTextSerializer.cpp#605 such that it's not just based on the tag name, but also/instead the display style? Maybe something with display: table-cell; should work the same as a td/th element?
(In reply to Nicolas Chevobbe [:nchevobbe] from comment #2)
> Yeah, I tried reverting back to a plain table with sticky th at some point, but that wasn't working well.

I think my patch is working pretty well, but there may be a slight difference in the borders. That might also be fixable though.

It's perhaps worth noting that Chrome doesn't have fixed headers - maybe it's not needed?

> No idea how the grid gets translated into a string when copying (this has always been a very consuming thing to get right)

Maybe an alternative solution is to change the logic here https://dxr.mozilla.org/mozilla-central/source/dom/base/nsPlainTextSerializer.cpp#605 such that it's not just based on the tag name, but also/instead the display style? Maybe something with display: table-cell; should work the same as a td/th element?

Back to Bug 1572777 Comment 3