Closed Bug 279031 Opened 21 years ago Closed 20 years ago

Reflect UI changes introduced with bug 231428 and bug 279034 in Help

Categories

(Documentation Graveyard :: Help Viewer, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Stefan.Borggraefe, Assigned: Stefan.Borggraefe)

References

Details

Attachments

(1 file, 1 obsolete file)

I will also make the necessary documentation changes for bug 279034 via this bug.
Depends on: 279034
Summary: Reflect UI changes introduced with bug 231428 in Help → Reflect UI changes introduced with bug 231428 and bug 279034 in Help
Attached patch Patch (obsolete) — Splinter Review
This patch also corrects some minor mistakes in the help texts for the two pref pages. I had to restructure the descriptions a bit to fit the new options in (using two levels of <ul> now, one for the groupbox captions and one for the specific options).
Attachment #173292 - Flags: review?(neil.parkwaycc.co.uk)
The text should say "a very large number of seconds". I don't like your sublists, but I don't know what we do elsewhere.
Should "quoted printable" be using "? Isn't there a <something> equivalent we use in help now?
(In reply to comment #4) > Should "quoted printable" be using "? Isn't there a <something> equivalent we > use in help now? We're using <q> for quotes.
Attached patch Patch V1.1Splinter Review
Ok, I corrected the spotted mistakes and removed two of the <ul>s I introduced with the first patch. I left the <ul> for the "Plain Text Messages"-part in to make it clear which part of the text only related to such messages. The others weren't really required. I also corrected stefan_h's nit from bug 277321 comment 9. ;-)
Attachment #173292 - Attachment is obsolete: true
Attachment #174001 - Flags: review?(neil.parkwaycc.co.uk)
Attachment #173292 - Flags: review?(neil.parkwaycc.co.uk)
I'm still pondering over the first sentence: >+<li><strong>Automatically save the message every ___ minutes</strong>: >+Choose this option if you want Mail &amp; Newsgroups to save the >+message you are currently composing automatically in the given >+interval. After a computer crash or program failure you can find the >+last saved version of the message in your Drafts folder.</li>
Comment on attachment 174001 [details] [diff] [review] Patch V1.1 OK, but I think it needs to say "at the given interval" and "latest saved version".
Attachment #174001 - Flags: review?(neil.parkwaycc.co.uk) → review+
Comment on attachment 174001 [details] [diff] [review] Patch V1.1 Thanks for the review. :-) I will make the suggested corrections before the check-in. Asking for approval. This is just a documentation update for some new functionalitiy that was added during the 1.8b1 cycle.
Attachment #174001 - Flags: approval1.8b?
Comment on attachment 174001 [details] [diff] [review] Patch V1.1 a=caillon for 1.8beta checkin
Attachment #174001 - Flags: approval1.8b? → approval1.8b+
Checking in extensions/help/resources/locale/en-US/mail_help.xhtml; /cvsroot/mozilla/extensions/help/resources/locale/en-US/mail_help.xhtml,v <--mail_help.xhtml new revision: 1.54; previous revision: 1.53 done
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: