Feature request: right-to-left (RTL) support for Arabic documents
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- react, tailwindcss, typescript
Línea de trabajo
Start in packages/ui/components/BlockRenderer.tsx, which the request names as the place to set a per-block dir; the same treatment is needed in BlockquoteBlock.tsx and TableBlock.tsx. Read the block prop shape there to see where block.content is available, then implement the RTL-vs-Latin counting rule from the issue and swap physical classes for logical ones (marginInlineStart, border-s-2 ps-4, text-start). Done means Arabic lists, blockquotes and tables flip in plan review while English output is byte-for-byte unchanged; there is no test named in the payload, so verify in the annotate and plan-review views.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What happens
Plans and documents written in Arabic (and other RTL languages) render left-to-right in annotate and plan review (plannotator 0.28.35, Windows, Claude Code):
- Arabic paragraphs and headings are left-aligned, and their punctuation jumps to the wrong end of the line.
- Bullets and numbers sit on the left of Arabic list items, and nested items indent from the left.
- The blockquote rule is on the left side.
- Table columns run left-to-right, and headers are forced left (
text-left). - The table of contents in the left sidebar has the same problems.
- Inline code inside an Arabic sentence (for example
Use `POST /ingest` herewritten in Arabic) splits the surrounding punctuation.
Agent output in Arabic is common now. The plans we review are mostly Arabic with English identifiers and code mixed in, so this affects every review.
Why dir="auto" is not enough
dir="auto" picks the direction from the first strong character. In mixed documents many Arabic lines start with an English identifier or code, for example a list item that begins with `POST /ingest` and continues in Arabic. Those lines would wrongly come out LTR. Short English-only headings inside an Arabic document (for example "Jev" or "WebSocket") also read better when they follow the document's direction.
Proposed change
-
A direction per block. For each block, take
block.content, drop the inline code spans, and compare the count of RTL letters with Latin letters:- The block is
rtlwhen RTL letters are at least half the Latin count. - The block is
ltrwhen it has only Latin text and at least about 30 letters. - Otherwise it follows the document's majority direction.
Set the result as a
dirattribute on the block root inpackages/ui/components/BlockRenderer.tsx(headings, paragraphs, list items),BlockquoteBlock.tsxandTableBlock.tsx. Then lists, tables and alignment follow from the browser's own bidi handling. - The block is
-
Logical properties instead of physical ones (Tailwind 4 supports them):
- list indent
style={{ marginLeft }}becomesmarginInlineStart; - blockquote
border-l-2 pl-4becomesborder-s-2 ps-4; - table headers
text-leftbecometext-start.
- list indent
-
Code stays LTR. Code blocks get
dir="ltr", and inlinecodegetsunicode-bidi: isolateso punctuation around it stays in place. -
Comments. The comment textarea gets
dir="auto", and the TOC entries use the same direction as their heading.
No text changes, so annotation offsets and quotes are unaffected. It also adds no setting: English documents keep rendering exactly as they do today.
Optional: a textDirection: "auto" | "ltr" | "rtl" setting for people who want to force one.
Workaround in the meantime
I'm using a small userscript that applies the same idea from outside: it sets dir per [data-block-id] and adds a few CSS overrides. It works, but it relies on internal class names and inline styles, so it breaks whenever the markup changes. A built-in fix would be much more robust.
I'm happy to open a PR along these lines if this direction works for you.
- Lenguaje dominante
- TypeScript
- Estrellas
- 8.7k
- Forks
- 649
- Merge medio
- 11 h 48 min
- PR fusionados (30 d)
- 552
Preparar el entorno
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de backnotprop/plannotator
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
backnotprop/plannotator#2317 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
backnotprop/plannotator#2306 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
backnotprop/plannotator#2290 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
backnotprop/plannotator#2243 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Annotate documentation omits per-file version history and privacy controlsQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
backnotprop/plannotator#1086 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de backnotprop/plannotator
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Graphify-Labs/graphify#4860 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
sveltejs/eslint-plugin-svelte#1615 ·
Los mantenedores suelen responder en 7 días
-
Parallel reticle_verify opens per-flow leases without the source lease's ignoreHTTPSErrorsPosiblemente ocupada @as992949791 la tomó hoy. Abiertogood first issue help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
reticlehq/reticle#2033 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día