https://bugs.documentfoundation.org/show_bug.cgi?id=164907
Justin L <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |NEW Whiteboard| |compatibilityMode15 Ever confirmed|0 |1 --- Comment #2 from Justin L <[email protected]> --- (In reply to Justin L from comment #0) > before we were pretty close to per-pixel > perfection) That is a bit of an exaggeration, but we did happen to be a whole lot closer... First - ignore the horizontal shift and table width difference. The table is 102%, and we simply can't handle that. Another thing rather unique about this document is that only the first row follows the TABLE's cell margins. The other rows have their own custom border spacing. We can't (really) handle that either. (reported as bug 165424) However, this document has me pretty convinced that comment 0's patch is simply, fundamentally wrong, and probably should just be reverted. 1.) we can't trust what the box says its border width or border spacing are. 2.) IIUC, this patch was trying to only add the BORDER width. However, it is actually adding both the top border and the top border spacing. [To add the border alone would use CalcLineWidth().] So in the case of this document, it is adding the border padding (0.5cm or 284) and the hairline border (10), which is why the rows are so much taller than they should be. 3.) Based on some testing I have done, it suggest that both the top and bottom borders are "added" to the spacing for atLeast heights, even in our IsCollapsingBorders() case. -- You are receiving this mail because: You are the assignee for the bug.
