Verify each finding against the current code and only fix it if needed.
Inline comments:
In
`@src/layout/RepeatingGroup/Summary2/RepeatingGroupTableSummary/RepeatingGroupTableSummary.test.tsx`:
- Around line 35-41: The test's rowsAfter fixture has three cells but the
rendered table columns (tableHeaders: ['input3']) only expect one data cell plus
an edit column; update the rowsAfter entry in
RepeatingGroupTableSummary.test.tsx to provide a single data cell (e.g., cells:
[{ text: 'summary.total' }]) so it matches the component (which appends the
extra edit cell when !pdfModeActive), and change the assertion to only check for
'summary.total' in the rendered output; reference the withRowsAfter/rowsAfter
test data and pdfModeActive behavior in the RepeatingGroupTableSummary component
when making this change.
In
`@src/layout/RepeatingGroup/Summary2/RepeatingGroupTableSummary/RepeatingGroupTableSummary.tsx`:
- Around line 92-103: rowsAfter rows rendered in RepeatingGroupTableSummary are
missing the extra trailing Table.Cell that DataRow always adds for the edit
button when pdfModeActive is false, causing column misalignment; update the
rowsAfter mapping in RepeatingGroupTableSummary (the block that maps rowsAfter
to Table.Row/Table.Cell) to append one additional empty <Table.Cell> when
pdfModeActive is false (match the same conditional used by DataRow/visibleIds
logic) so each rowsAfter row has visibleIds.length + 1 cells and aligns with the
header and DataRow entries.
---
Nitpick comments:
In
`@src/layout/RepeatingGroup/Summary2/RepeatingGroupTableSummary/RepeatingGroupTableSummary.tsx`:
- Around line 96-99: The extra !== undefined check after 'text' in cell is
likely redundant; inspect the cell/rowsAfter type used by
RepeatingGroupTableSummary to confirm whether the text property is declared
optional; if text is non-optional then remove the "&& cell.text !== undefined"
from the Lang render line (leaving {cell && 'text' in cell && <Lang
id={cell.text} />), otherwise if the type is optional keep the explicit
undefined check or prefer a nullish check (cell.text != null) to be explicit;
update the RepeatingGroupTableSummary component accordingly (referencing the
cell variable and the Lang and ComponentSummary render branches).

Verify each finding against the current code and only fix it if needed.
Inline comments:
In
`@src/layout/RepeatingGroup/Summary2/RepeatingGroupTableSummary/RepeatingGroupTableSummary.tsx`:
- Around line 92-104: rowsAfter rows currently render every entry in row.cells
and can produce a different cell count than DataRow (which only renders
visibleIds based on columnSettings[id]?.hidden), causing misalignment; update
the rowsAfter rendering in RepeatingGroupTableSummary to filter row.cells the
same way DataRow does—either map visibleIds to pick the corresponding cell
entries (using the same visibleIds and columnSettings logic) or apply the
hiddenColumnIndices approach used by GridRowsRenderer/GridRowRenderer so
rowsAfter outputs exactly visibleIds.length cells (plus the extra cell when
!pdfModeActive).

Verify each finding against the current code and only fix it if needed.
Duplicate comments:
In
`@src/layout/RepeatingGroup/Summary2/RepeatingGroupTableSummary/RepeatingGroupTableSummary.tsx`:
- Around line 95-110: The previous alignment issue is resolved: ensure no code
changes are needed — keep the filtering on rowsAfter using hiddenColumnIndices
and preserve the trailing empty cell condition using !pdfModeActive && !isSmall
so RepeatingGroupTableSummary (rowsAfter rendering) matches the header behavior;
verify the logic around rowsAfter, hiddenColumnIndices, pdfModeActive and
isSmall remains as shown and do not revert to DataRow's !pdfModeActive
condition.