Closed Bug 279031 Opened 20 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: