Skip to content

fix: anchor table overlaps text#1995

Merged
caio-pizzol merged 2 commits intomainfrom
sd-1679_anchored-table
Feb 12, 2026
Merged

fix: anchor table overlaps text#1995
caio-pizzol merged 2 commits intomainfrom
sd-1679_anchored-table

Conversation

@VladaHarbour
Copy link
Contributor

Tables that have w:tblpPr devined are treated as anchor tables and can overlap the text
Before:
Screenshot 2026-02-11 at 14 46 56
After:
Screenshot 2026-02-11 at 14 46 33

@linear
Copy link

linear bot commented Feb 11, 2026

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d55da94c54

ℹ️ About Codex in GitHub

Your team has set up Codex to 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 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Contributor

@caio-pizzol caio-pizzol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@VladaHarbour this is great!

One thing to note here: since this touches rendering of documents, I think it's worth uploading a sample file or the file that caused the issue to our visual testing suite

(let me know if you want to sync on how to do that)

@caio-pizzol caio-pizzol linked an issue Feb 12, 2026 that may be closed by this pull request
@caio-pizzol caio-pizzol requested a review from tupizz February 12, 2026 15:03
Copy link
Contributor

@luccas-harbour luccas-harbour left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@tupizz tupizz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@VladaHarbour VladaHarbour force-pushed the sd-1679_anchored-table branch from d55da94 to f794f41 Compare February 12, 2026 21:21
@github-actions
Copy link
Contributor

Visual diffs detected

Pixel differences were found in visual tests. This is not blocking — reproduce locally with cd tests/visual && pnpm docs:download && pnpm test to review diffs.

@caio-pizzol caio-pizzol merged commit fc05e29 into main Feb 12, 2026
4 checks passed
@caio-pizzol caio-pizzol deleted the sd-1679_anchored-table branch February 12, 2026 21:44
@superdoc-bot
Copy link

superdoc-bot bot commented Feb 12, 2026

🎉 This PR is included in superdoc v1.13.0-next.3

The release is available on GitHub release

superdoc-bot bot pushed a commit that referenced this pull request Feb 14, 2026
# [1.13.0](v1.12.0...v1.13.0) (2026-02-14)

### Bug Fixes

* anchor table overlaps text ([#1995](#1995)) ([fc05e29](fc05e29))
* collaboration cursor styles fix ([fd6db10](fd6db10))
* ensure we do not duplicate bubble text ([#1934](#1934)) ([c41cf9e](c41cf9e))
* ignore sdBlockId when pasting content ([#2010](#2010)) ([1b08572](1b08572))
* normalize bookmarks in tables ([#1892](#1892)) ([369b7e1](369b7e1))
* replace Node.js Buffer APIs with browser-native alternatives ([#2028](#2028)) ([b17774a](b17774a)), closes [#exportProcessMediaFiles](https://github.com/superdoc-dev/superdoc/issues/exportProcessMediaFiles)
* use correct template syntax for GitHub release URL in PR comments ([9d1bca2](9d1bca2))

### Features

* enabled telemetry by default and added documentation ([#2001](#2001)) ([8598ef7](8598ef7))
* enhance telemetry handling for sub-editors ([#2017](#2017)) ([37bc030](37bc030))
* update telemetry configuration to prioritize root licenseKey ([#2016](#2016)) ([3b4ff6b](3b4ff6b))
@superdoc-bot
Copy link

superdoc-bot bot commented Feb 14, 2026

🎉 This PR is included in superdoc v1.13.0

The release is available on GitHub release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Table overlaps with text

4 participants