fix(rtl): set dir=ltr on identity <option> elements

The From-identity picker in the composer and the template form use a
native <select>, so the earlier <bdi> fix can't apply there - browsers
render <option> as plain text and strip any nested markup. The native
OS-rendered option list still respects the dir attribute directly
though, so setting dir="ltr" on each option fixes the same bracket-
mirroring bug for "Name <email>" entries in that native popup.
This commit is contained in:
xhzeem
2026-07-21 23:08:02 +02:00
committed by Linus Rath
parent c7250dc921
commit 23a017d4b7
2 changed files with 3 additions and 3 deletions
+2 -2
View File
@@ -2192,7 +2192,7 @@ export function EmailComposer({
? generateSubAddress(identity.email, subAddressTag, subAddressDelimiter) ? generateSubAddress(identity.email, subAddressTag, subAddressDelimiter)
: identity.email; : identity.email;
return ( return (
<option key={identity.id} value={identity.id}> <option key={identity.id} value={identity.id} dir="ltr">
{identity.name ? `${identity.name} <${displayEmail}>` : displayEmail} {identity.name ? `${identity.name} <${displayEmail}>` : displayEmail}
</option> </option>
); );
@@ -2204,7 +2204,7 @@ export function EmailComposer({
? generateSubAddress(identity.email, subAddressTag, subAddressDelimiter) ? generateSubAddress(identity.email, subAddressTag, subAddressDelimiter)
: identity.email; : identity.email;
return ( return (
<option key={identity.id} value={identity.id}> <option key={identity.id} value={identity.id} dir="ltr">
{identity.name ? `${identity.name} <${displayEmail}>` : displayEmail} {identity.name ? `${identity.name} <${displayEmail}>` : displayEmail}
</option> </option>
); );
+1 -1
View File
@@ -242,7 +242,7 @@ export function TemplateForm({ template, initialData, onSave, onCancel }: Templa
> >
<option value="">{tSettings('default_identity')}</option> <option value="">{tSettings('default_identity')}</option>
{identities.map((id) => ( {identities.map((id) => (
<option key={id.id} value={id.id}> <option key={id.id} value={id.id} dir="ltr">
{id.name ? `${id.name} <${id.email}>` : id.email} {id.name ? `${id.name} <${id.email}>` : id.email}
</option> </option>
))} ))}