Skip to content

feat(core): support RTL tables with w:bidiVisual alignment#800

Open
Pallavikumarimdb wants to merge 1 commit into
eigenpal:mainfrom
Pallavikumarimdb:fix/rtl-table-bidivisual
Open

feat(core): support RTL tables with w:bidiVisual alignment#800
Pallavikumarimdb wants to merge 1 commit into
eigenpal:mainfrom
Pallavikumarimdb:fix/rtl-table-bidivisual

Conversation

@Pallavikumarimdb

@Pallavikumarimdb Pallavikumarimdb commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Summary

Closes: #734 by adding support for RTL tables containing <w:bidiVisual/>.

Changes

  • Added bidi attribute to the table node attributes interface and ProseMirror schema.
  • Implemented logic in toDOM to apply dir="rtl" and style direction: rtl to tables when bidi is true, ensuring correct visual rendering of columns.
  • Updated mapping logic in toProseDoc and fromProseDoc to parse and serialize <w:bidiVisual/> to/from the document model.
  • Added unit tests to cover parsing, rendering, and round-tripping of RTL tables.

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@vercel

vercel Bot commented Jun 14, 2026

Copy link
Copy Markdown

@Pallavikumarimdb is attempting to deploy a commit to the EigenPal Team on Vercel.

A member of the Team first needs to authorize it.

@eigenpal-release-pal

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅

Posted by the CLA bot.

@Pallavikumarimdb Pallavikumarimdb marked this pull request as ready for review June 14, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RTL table ignores w:bidiVisual, causing Hebrew label to appear on wrong side of line

1 participant