Skip to content

Conversation

@yash-agarwa-l
Copy link
Contributor

This fixes the layout of block quotes (> quote) in RTL languages.

Previously, the vertical bar was stuck on the Left side even in RTL mode. Now, it correctly flips to the Right side (the "start" of the text).

This addresses one of the RTL issues identified by @gnprice in this comment.

Before

LTR

LTR

RTL

RTL

After

LTR

LTR

RTL

RTL

@gnprice gnprice added the maintainer review PR ready for review by Zulip maintainers label Nov 25, 2025
Previously, the `Quotation` widget hardcoded the border and padding
to the left side. This resulted in an incorrect layout in RTL
locales, where the quote bar remained on the left while the text
aligned to the right.

We now use `EdgeInsetsDirectional` and `BorderDirectional` so that
the layout flips correctly based on the text direction.
@chrisbobbe chrisbobbe self-requested a review December 2, 2025 20:07
@chrisbobbe chrisbobbe self-assigned this Dec 2, 2025
Copy link
Collaborator

@chrisbobbe chrisbobbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks! Marking for Greg's review.

@chrisbobbe chrisbobbe requested a review from gnprice December 2, 2025 20:08
@chrisbobbe chrisbobbe assigned gnprice and unassigned chrisbobbe Dec 2, 2025
@chrisbobbe chrisbobbe added integration review Added by maintainers when PR may be ready for integration and removed maintainer review PR ready for review by Zulip maintainers labels Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

integration review Added by maintainers when PR may be ready for integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants