Skip to content

Support right-to-left (Arabic/Hebrew) reports - #45

Merged
aihamtaleb merged 1 commit into
mainfrom
fix/arabic-rtl-support
Sep 28, 2026
Merged

aihamtaleb merged 1 commit into
mainfrom
fix/arabic-rtl-support

Conversation

@aihamtaleb

Copy link
Copy Markdown
Contributor
  • Report panel: make section ids Unicode-aware and unique. The ASCII-only slug collapsed every Arabic heading to "-", so duplicate React keys made editor updates render as appended sections instead of replacing placeholders in place.
  • Frontend: set dir="rtl" on report and chat markdown when RTL letters dominate, and switch physical left/right classes to logical ones.
  • PDF: emit dir="rtl" for RTL reports and mirror indents, list padding and blockquote borders.
  • Charts: send a self-contained Arabic shaping + bidi reordering module to the Code Interpreter sandbox (matplotlib 3.9, no network) ahead of the agent's code; it is a no-op on matplotlib >= 3.11, which handles RTL.

Issue #, if available:

Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

- Report panel: make section ids Unicode-aware and unique. The ASCII-only
  slug collapsed every Arabic heading to "-", so duplicate React keys made
  editor updates render as appended sections instead of replacing
  placeholders in place.
- Frontend: set dir="rtl" on report and chat markdown when RTL letters
  dominate, and switch physical left/right classes to logical ones.
- PDF: emit dir="rtl" for RTL reports and mirror indents, list padding
  and blockquote borders.
- Charts: send a self-contained Arabic shaping + bidi reordering module to
  the Code Interpreter sandbox (matplotlib 3.9, no network) ahead of the
  agent's code; it is a no-op on matplotlib >= 3.11, which handles RTL.
@aihamtaleb
aihamtaleb merged commit 33bf68e into main Sep 28, 2026
3 of 4 checks passed
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.

1 participant