Skip to content

Commit 4758ffa

Browse files
ars18wrwiText-CI
authored andcommitted
Remove outdated TODOs in TableRenderer
childRenderers are already cleared at the beginning of #layout() currentBorder is considered only for collapsed borders DEVSIX-4164 Autoported commit. Original commit hash: [a878839cb]
1 parent a743323 commit 4758ffa

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

itext/itext.layout/itext/layout/renderer/TableRenderer.cs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1241,8 +1241,6 @@ protected internal virtual iText.Layout.Renderer.TableRenderer CreateSplitRender
12411241
splitRenderer.rowRange = rowRange;
12421242
splitRenderer.parent = parent;
12431243
splitRenderer.modelElement = modelElement;
1244-
// TODO childRenderers will be populated twice during the relayout.
1245-
// We should probably clean them before #layout().
12461244
splitRenderer.childRenderers = childRenderers;
12471245
splitRenderer.AddAllProperties(GetOwnProperties());
12481246
splitRenderer.headerRenderer = headerRenderer;
@@ -1548,7 +1546,7 @@ private void CorrectLayoutedCellsOccupiedAreas(LayoutResult[] splits, int row, i
15481546
// correct last height
15491547
int finish = bordersHandler.GetFinishRow();
15501548
bordersHandler.SetFinishRow(rowRange.GetFinishRow());
1551-
// TODO Correct for collapsed borders only
1549+
// It's width will be considered only for collapsed borders
15521550
Border currentBorder = bordersHandler.GetWidestHorizontalBorder(finish + 1);
15531551
bordersHandler.SetFinishRow(finish);
15541552
if (skip) {

port-hash

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
659fc60b0ae4bf6b81c0e41173723b2b5316dc61
1+
a878839cb0dc540fcec4bd539bc66505c7e7297b

0 commit comments

Comments
 (0)