Skip to content

feat: rtl for tables#3227

Open
artem-harbour wants to merge 1 commit into
mainfrom
artem/rtl-v6
Open

feat: rtl for tables#3227
artem-harbour wants to merge 1 commit into
mainfrom
artem/rtl-v6

Conversation

@artem-harbour
Copy link
Copy Markdown
Contributor

@artem-harbour artem-harbour commented May 11, 2026

Linear: SD-2810

Phase 5 (RTL Table Stability) is now completed for RTL table interaction and roundtrip reliability.

This PR does not introduce RTL tables from scratch; core support was already in place.
The main work here was stabilization and regression coverage:

  • Confirmed full bidiVisual pipeline behavior (<w:bidiVisual/>tableProperties.rightToLeft)
  • Validated RTL table rendering order and border behavior in the current layout pipeline
  • Verified click-to-position behavior for RTL tables, including empty/padding areas
  • Verified Tab and Shift+Tab table navigation behavior against Word
  • Confirmed export roundtrip preserves RTL table semantics without border-side corruption
  • Fixed RTL resize/drag boundary mapping issues, including handle-to-column targeting and post-resize hover indicator consistency
  • Added regression tests for RTL and LTR table resize/hover flows
  • Added focused behavior and import/export tests using real DOCX fixtures

Overall result: RTL tables are now covered by explicit tests across rendering, interaction, and export paths, with resize/drag behavior stabilized and regression-protected.

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@linear
Copy link
Copy Markdown

linear Bot commented May 11, 2026

SD-2810

@artem-harbour artem-harbour marked this pull request as ready for review May 11, 2026 17:39
@artem-harbour artem-harbour requested a review from a team as a code owner May 11, 2026 17:39
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.

2 participants