Don't down-convert message to plain text if dir attribute is set

RESOLVED FIXED in mozilla1.7alpha

Status

()

Core
Layout: Text
RESOLVED FIXED
14 years ago
10 years ago

People

(Reporter: Prognathous, Assigned: Scott MacGregor)

Tracking

Trunk
mozilla1.7alpha
Points:
---
Bug Flags:
blocking1.6 +

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

14 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6b) Gecko/20031212
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.6b) Gecko/20031212

Mozilla Format Auto-Detection wrongly converts HTML email with dir="rtl" to
plain text. This results in a left-aligned, LTR message - a serious problem for
Hebrew and Arabic users.

Reproducible: Always

Steps to Reproduce:
1. Launch Mozilla Mail&News and press Compose.
2. Change the body direction to of the mail to RTL.
3. Type some Hebrew or Arabic characters followed by a full stop.
4. Send the mail.

* To change the body to RTL, focus the caret on the body of the mail -> Format
menu -> Page colors and Backgournd -> Advanced Edit -> Attribute Dir=rtl -> OK -> OK

Actual Results:  
The mail is converted to plain text, losing the direction attribute. The
recipient sees an LTR message, with the full stop at the wrong side (rightmost)
and wrong word order if some LTR words are included.

-> Content-Type: text/plain; charset=windows-1255; format=flowed


Expected Results:  
The mail should remain HTML (preferably, multipart HTML+plain text) in order to
retain the direction. 


If "strong" formatting elements are included the mail does get sent as text/html
(not multipart).

Prog.
(Reporter)

Comment 1

14 years ago
Can someone please confirm this bug and reassign it to Scott MacGregor
(mscott@mozilla.org)? He already provides a patch for this issue in the related
Bug 136502.

Thanks!

Prog.
(Reporter)

Updated

14 years ago
Assignee: mkaply → mscott
(Reporter)

Updated

14 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true
(Assignee)

Comment 2

14 years ago
Patch checked in. r/sr=bienvenu. The patch was in 136502.
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.7alpha
(Reporter)

Comment 3

14 years ago
Scott, how about asking approval1.6 for this fix? It definely fits my definition
of "high reward, low risk change".

Later today, I'm going to deploy a build of the current 1.6 branch to quite a
few testers. This build will include the patch for bug 162242, and it would be
great to also apply your patch for this bug. If no regressions are found, then
the fixes for both bugs would make 1.6 a huge release for BiDi users.

Prog.

Comment 4

14 years ago
the build prog made can be found here:
http://www.xslf.com/mozilla_bin.exe
(Assignee)

Comment 5

14 years ago
this has been checked into the 1.6 branch
Flags: blocking1.6+
Product: MailNews → Core

Updated

10 years ago
Component: MailNews: BiDi Hebrew & Arabic → Layout: Text
QA Contact: giladehven → layout.fonts-and-text
You need to log in before you can comment on or make changes to this bug.