Skip to content

Generated by Codex: Preserve centered markdown with RTL block direction#60

Draft
omers-oai wants to merge 8 commits intomainfrom
codex/rtl-markdown-block-alignment-v2
Draft

Generated by Codex: Preserve centered markdown with RTL block direction#60
omers-oai wants to merge 8 commits intomainfrom
codex/rtl-markdown-block-alignment-v2

Conversation

@omers-oai
Copy link

Generated by Codex:

Summary

  • keep the RTL markdown block fix by preserving per-paragraph text direction
  • stop forcing left/right paragraph alignment so centered ambient text styles still work
  • update the paragraph-direction regression test to match the smaller API surface

Testing

  • Parallel Configuration Cache is an incubating feature.
    Configuration on demand is an incubating feature.
    Calculating task graph as no cached configuration is available for tasks: :richtext-markdown:allTests :richtext-ui:jvmTest
    Type-safe project accessors is an incubating feature.

[Incubating] Problems report is available at: file:///Users/omers/.codex/worktrees/aa49/openai-android/build/reports/problems/problems-report.html

Publishing Build Scan to Develocity...
https://gradle.com/s/ejen2t5dmsrc6

Configuration cache entry stored.

@omers-oai omers-oai marked this pull request as ready for review March 17, 2026 00:45
@tananaev
Copy link
Member

Does this solve the problem with expanding width? Do you have some preview comparison to validate?

@omers-oai
Copy link
Author

Yes, it should. I'm always kind of stuck with one thread pulling another here, but recent tests on the new commit in the main app showed it working well.

I'll merge this when testing and snapshots are done on the main upgrade.

@omers-oai omers-oai force-pushed the codex/rtl-markdown-block-alignment-v2 branch from e3d92d1 to 605cadb Compare March 17, 2026 20:30
@omers-oai omers-oai marked this pull request as draft March 20, 2026 00:48
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.

2 participants