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)
Documentation Graveyard
Help Viewer
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Stefan.Borggraefe, Assigned: Stefan.Borggraefe)
References
Details
Attachments
(1 file, 1 obsolete file)
7.32 KB,
patch
|
neil
:
review+
caillon
:
approval1.8b+
|
Details | Diff | Splinter Review |
Assignee | ||
Comment 1•20 years ago
|
||
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
Assignee | ||
Comment 2•20 years ago
|
||
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)
Comment 3•20 years ago
|
||
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?
Comment 5•20 years ago
|
||
(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.
Assignee | ||
Comment 6•20 years ago
|
||
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)
Assignee | ||
Updated•20 years ago
|
Attachment #173292 -
Flags: review?(neil.parkwaycc.co.uk)
Comment 7•20 years ago
|
||
I'm still pondering over the first sentence:
>+<li><strong>Automatically save the message every ___ minutes</strong>:
>+Choose this option if you want Mail & 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 8•20 years ago
|
||
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+
Assignee | ||
Comment 9•20 years ago
|
||
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 10•20 years ago
|
||
Comment on attachment 174001 [details] [diff] [review] Patch V1.1 a=caillon for 1.8beta checkin
Attachment #174001 -
Flags: approval1.8b? → approval1.8b+
Assignee | ||
Comment 11•20 years ago
|
||
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.
Description
•