Closed Bug 37967 Opened 25 years ago Closed 24 years ago

Hook up HTML and Plain Text domains pref section

Categories

(MailNews Core :: Backend, defect, P2)

defect

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.8

People

(Reporter: laurel, Assigned: bugzilla)

References

Details

(Whiteboard: [nsbeta1+])

Attachments

(1 file)

Mail pref spec: http://gooey/client/5.0/specs/mail/Prefs/Preferences.html#Sending According to the 4/27/00 revision of the mail preference spec, the Sending Format (formerly known as Formatting) pref panel will have an html/plain text domains section. This bug is to track the hookup of that section to html intelligent send feature functionality. Bug to add the section to pref UI is logged as bug #37928. Standard Disclaimer: check spec before implementing for up-to-date revisions.
QA Contact: lchiang → fenella
Sounds like Jean-Francois work......let me know if you think I should find someone else Jean-Francois...
Assignee: mscott → ducarroz
Keywords: nsbeta2
yep, it's mine. But I doubt I will have enough time to do it for B2!
Status: NEW → ASSIGNED
Target Milestone: --- → M17
Putting on [nsbeta2+][5/16] radar. This is a feature MUST complete work by 05/16 or we may pull this feature for PR2.
Whiteboard: [nsbeta2+][5/16]
Putting on [nsbeta2-] radar. Missed the Netscape 6 feature train. Please set to MFuture.
Whiteboard: [nsbeta2+][5/16] → [nsbeta2-]
Moving to future milestone.
Target Milestone: M17 → Future
Spec is at <http://www.mozilla.org/mailnews/specs/prefs/Preferences.html>. J-F, don't you have that ready to check in, according to the latest status report? Where is the patch? Why is it future? Is this a dup?
good catch, moving to moz08 and nominating nsbeta1. The UI which is covered by bug 37928 is ready to checkin, I am just waiting on candice to finish the AB part (see bug 42038) before I can finish this bug and then checkin all the pieces.
Depends on: 37928, 42038
Keywords: nsbeta2nsbeta1
Whiteboard: [nsbeta2-]
Target Milestone: Future → mozilla0.8
Where is the patch for the backend?
yep we need a bug to track this. marking nsbeta1+
Priority: P3 → P2
Whiteboard: [nsbeta1+]
this needs the front end to get done and that won't land until mozilla0.9, so moving this but to mozilla0.9
Target Milestone: mozilla0.8 → mozilla0.9
Assign to Sheela
QA Contact: fenella → sheelar
Has I've already a fix for this bug, I move it back to moz0.8
Whiteboard: [nsbeta1+] → [nsbeta1+] Fix in hand
Target Milestone: mozilla0.9 → mozilla0.8
Attached patch proposed fix (deleted) — Splinter Review
sr=mscott
r=chuang
you could use an nsXPIDLString here: PRUnichar *str; + prefs->CopyUnicharPref("mailnews.plaintext_domains", &str); + plaintextDomains = str; which would clean up the code a little.
Other than that, sr=bienvenu.
There is goes: if (prefs) { nsXPIDLString str; prefs->CopyUnicharPref("mailnews.plaintext_domains", getter_Copies(str)); plaintextDomains = str; prefs->CopyUnicharPref("mailnews.html_domains", getter_Copies(str)); htmlDomains = str; }
Fixed and checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
Whiteboard: [nsbeta1+] Fix in hand → [nsbeta1+]
reassign to esther
QA Contact: sheelar → esther
Using builds dated 03-05-01 on winme, mac and linux the backend and frontend are in and basically working. Needs some clean up after testing. Will log new bugs. 1st new bug is the default size of the window.
Status: RESOLVED → VERIFIED
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: