Closed Bug 484143 Opened 13 years ago Closed 13 years ago

Remove >> AOLCiter <<

Categories

(Core :: DOM: Editor, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla1.9.2a1

People

(Reporter: philor, Assigned: rain1)

Details

Attachments

(2 files, 1 obsolete file)

Attached patch Naive patch (obsolete) — Splinter Review
The plaintext editor offers an option, based on the pref mail.compose.citationType, to use what was apparently AOL's style of quoting in mail some time before 1999, wrapping a quote in ">> " and " <<".

While we do actually recognize that as a quote in composition, at least to the extent of showing it in blue text, even we don't know what to make of it when we receive it, treating the first line as being two quote-levels deep and the rest as unquoted.

The world will be a better place once it's wiped from the face of the earth (and editor will have a little less ugly twisted code, and slightly fewer unnecessary includes, to boot).
roc -- is it fine if you review this?
Assignee: nobody → sid.bugzilla
Attachment #368204 - Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #371151 - Flags: superreview?
Attachment #371151 - Flags: review?
Attachment #371151 - Flags: superreview?(roc)
Attachment #371151 - Flags: superreview?
Attachment #371151 - Flags: review?(roc)
Attachment #371151 - Flags: review?
Comment on attachment 371151 [details] [diff] [review]
get rid of MakeACiter() as well

I assume you've checked that Thunderbird and Seamonkey don't have UI for this?
Attachment #371151 - Flags: superreview?(roc)
Attachment #371151 - Flags: superreview+
Attachment #371151 - Flags: review?(roc)
Attachment #371151 - Flags: review+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/f643cb847b6a
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
You need to log in before you can comment on or make changes to this bug.