fix: Preserve table cell styles on Google Docs paste by Branc0 · Pull Request #2628 · superdoc-dev/superdoc

Branc0 · 2026-03-30T00:48:34Z

Summary

PR for the issue Issue #2150.

Table cell styling is lost when pasting from Google Docs. Cells with background colors are pasted as plain white; borders and padding are dropped.

Additionally, as a side effect of the main fixes, this PR solves two other issues:

Google Docs Pasted tables don't persist row height: I had to fix to test vertical alignment.
Duplicated borders on the intersections of cells. As we make the borders bigger, this gets more evident.

Steps for testing

Create a table in Google Docs with colored cells (e.g., this doc ).
Copy the table and paste it into SuperDoc.
Table cell background colors, borders, padding, and vertical alignment from pasted content are preserved.

…eDocs

…ding tests

Branc0 · 2026-03-30T00:51:09Z

packages/super-editor/src/editors/v1/extensions/table-cell/helpers/parseCellMargins.js

+  const [bottomValuePt] = parseSizeUnit(style?.paddingBottom);
+  const [leftValuePt] = parseSizeUnit(style?.paddingLeft);
+
+  const top = halfPointToPixels(topValuePt);


I was searching for a helper function that converts PT to PX, and I noticed this has the same formula, so I used it despite the name being halfPoint. Let me know if it's okay...

@Branc0 the formula is right, just the name is confusing — we mentioned a shared helper in the inline review.

packages/super-editor/src/editors/v1/extensions/table-cell/table-cell.js

packages/super-editor/src/editors/v1/extensions/table-row/helpers/parseRowHeight.js

…egex pattern

packages/super-editor/src/editors/v1/extensions/table-cell/helpers/parseCellBorders.js

packages/layout-engine/painters/dom/src/table/renderTableRow.ts

chatgpt-codex-connector

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 5b77732f99

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

Open a pull request for review
Mark a draft as ready
Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

chatgpt-codex-connector · 2026-03-30T01:27:21Z

packages/layout-engine/painters/dom/src/table/renderTableRow.ts

+        right: cellBounds.touchesRightEdge ? resolveTableBorderValue(cellBorders.right, tableBorders.right) : undefined,
+        bottom: touchesBottomBoundary ? resolveTableBorderValue(cellBorders.bottom, tableBorders.bottom) : undefined,


Preserve explicit interior cell borders in collapsed mode

In resolveRenderedCellBorders, the collapsed-table branch drops right/bottom for every non-edge cell even when those sides are explicitly set on the cell. This means interior overrides (for example a custom right border on one cell) are silently replaced by table defaults or disappear entirely when insideV/insideH is absent, so pasted/imported tables lose intended border styling whenever tableBorders exists.

Useful? React with 👍 / 👎.

@Branc0 this is intentional — Google Docs always puts the same border on both sides, so nothing gets lost.

packages/super-editor/src/editors/v1/extensions/table-cell/helpers/parseCellMargins.js

Branc0 · 2026-03-30T01:50:15Z

Still WIP on this one:

The P1 issue from codex: tried a couple of fixes, but they introduced the duplicated border back
Integration test

caio-pizzol

@Branc0 tested with real Google Docs tables — backgrounds, borders (solid/dashed/dotted), merged cells, borderless cells, padding, column widths all working. no blockers.

test docs: basic styles · edge cases

basic styles — GDocs vs SuperDoc:

edge cases — GDocs vs SuperDoc:

left a few inline comments.

packages/super-editor/src/editors/v1/extensions/table-cell/helpers/parseCellBorders.js

packages/layout-engine/painters/dom/src/table/renderTableRow.ts

caio-pizzol · 2026-03-30T20:07:46Z

packages/super-editor/src/editors/v1/extensions/table-cell/helpers/parseCellMargins.js

@@ -0,0 +1,52 @@
+// @ts-check


re: halfPointToPixels — same formula is now in three files. a small shared helper would keep it in one place, worth doing before more paste handlers land?

@caio-pizzol, we briefly spoke about this on Friday morning, but I forgot the outcome of our discussion 😅.

halfPointToPixels is already coming from "@core/super-converter/helpers.js", do you want me to change its name, or keep it as it is?

keep the name, it's fine. the duplication is minor — not worth extracting for this PR.

caio-pizzol · 2026-04-02T20:56:49Z

Hey @Branc0, this PR has had no activity for 3 days.

…borders

…ollapsed mode

…ble cell styles

caio-pizzol

@Branc0 tests and comment look good, resolved those two threads.

one thing: Google Docs uses <th> for header rows, but table-header.js only has rendering — it's missing the parsing side for the new attributes. so when you paste a table with a header row, the header cells lose their padding (and would lose background/borders/alignment too if they had any). table-cell.js has both sides, table-header.js needs the same. tested with this doc.

re: the halfPointToPixels name — keep it as is, it's fine.

Branc0 added 7 commits March 29, 2026 19:40

fix(table-cell): add background color parsing from inline styles

3a6b16c

fix(table-cell): dd cell margin parsing from inline styles

10b70b8

fix(table-row): persists row height on pasted content from html

65f7fdc

fix(table-cell): persists cell verticallAlignment on paste from googl…

ba53f9f

…eDocs

fix(test): tests for color and margin were failing

1ba1a41

feat(table-cell): add border parsing from inline styles and correspon…

ab5e67b

…ding tests

fix(table-row): prevent double interior borders

4861006

superdoc-bot bot added review: thorough community labels Mar 30, 2026

Branc0 commented Mar 30, 2026

View reviewed changes

packages/super-editor/src/editors/v1/extensions/table-cell/table-cell.js Show resolved Hide resolved

Branc0 commented Mar 30, 2026

View reviewed changes

packages/super-editor/src/editors/v1/extensions/table-row/helpers/parseRowHeight.js Show resolved Hide resolved

refactor(table-cell): optimize border style parsing using a dynamic r…

5b77732

…egex pattern

Branc0 commented Mar 30, 2026

View reviewed changes

packages/super-editor/src/editors/v1/extensions/table-cell/helpers/parseCellBorders.js Show resolved Hide resolved

Branc0 commented Mar 30, 2026

View reviewed changes

packages/layout-engine/painters/dom/src/table/renderTableRow.ts Show resolved Hide resolved

Branc0 marked this pull request as ready for review March 30, 2026 01:21

chatgpt-codex-connector bot reviewed Mar 30, 2026

View reviewed changes

refactor(table-cell): returns margins with value only

27946cb

caio-pizzol self-assigned this Mar 30, 2026

caio-pizzol reviewed Mar 30, 2026

View reviewed changes

Branc0 added 6 commits April 5, 2026 12:56

test(table-cell): increase cooverage and refactor tests

199a112

docs(renderTableRow): clarify assumptions about shared interior cell …

27ad316

…borders

test(renderTableRow): add test for non-final row border behavior in c…

36fd587

…ollapsed mode

Merge branch 'main' into 2150-preserve-table-styles

661df8c

Merge branch 'main' into 2150-preserve-table-styles

82b9242

test(table-cell): add integration tests for HTML paste handling of ta…

f8a6982

…ble cell styles

caio-pizzol reviewed Apr 6, 2026

View reviewed changes

		right: cellBounds.touchesRightEdge ? resolveTableBorderValue(cellBorders.right, tableBorders.right) : undefined,
		bottom: touchesBottomBoundary ? resolveTableBorderValue(cellBorders.bottom, tableBorders.bottom) : undefined,

Conversation

Branc0 commented Mar 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Steps for testing

Uh oh!

Branc0 Mar 30, 2026

Choose a reason for hiding this comment

Uh oh!

caio-pizzol Mar 30, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

💡 Codex Review

Uh oh!

chatgpt-codex-connector bot Mar 30, 2026

Choose a reason for hiding this comment

Uh oh!

caio-pizzol Mar 30, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Branc0 commented Mar 30, 2026

Uh oh!

caio-pizzol left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

caio-pizzol Mar 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Branc0 Apr 5, 2026

Choose a reason for hiding this comment

Uh oh!

caio-pizzol Apr 6, 2026

Choose a reason for hiding this comment

Uh oh!

caio-pizzol commented Apr 2, 2026

Uh oh!

caio-pizzol left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Branc0 commented Mar 30, 2026 •

edited

Loading

caio-pizzol left a comment •

edited

Loading

caio-pizzol Mar 30, 2026 •

edited

Loading