Closed Bug 236060 Opened 21 years ago Closed 21 years ago

Shouldn't refer to "Mozilla menu on Mac" OS X in "System" section

Categories

(Documentation Graveyard :: Help Viewer, defect)

Product:
Documentation Graveyard
Component:
Help Viewer
Type:
defect
Priority:
Not set
Severity:
trivial

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: stefanh, Assigned: stefanh)

References

(
URL
)

Details

(Keywords: late-l10n)

Attachments

(1 file)

Patch
(deleted), patch
neil
: review+
mkaply
: approval1.7+
Details | Diff | Splinter Review
Missed/didn't see this when fixing bug 210335... (sigh)
Attached patch Patch (deleted) — — Splinter Review
Assignee: rlk → stefan_h
Status: NEW → ASSIGNED
Attachment #142643 - Flags: review?(rlk)
Why wouldn't this be Mozilla menu? Do we have preferences in two locations on MacOS?
i don't think we have system preferences under mac os
> i don't think we have system preferences under mac os We don't: Line 351: <p><b>Note</b>: The System preferences panel appears only on Windows- based platforms.</p> IMHO that could be made clearer :)
Comment on attachment 142643 [details] [diff] [review] Patch neil: can you review this? I'm not too experienced on mac and want to make sure this is correct.
Attachment #142643 - Flags: review?(rlk) → review?(neil.parkwaycc.co.uk)
Comment on attachment 142643 [details] [diff] [review] Patch The internal name for the system pref panel is winhooks :-) I do wonder why this uses 1. and 2. instead of <ol> and <li> though...
Attachment #142643 - Flags: review?(neil.parkwaycc.co.uk) → review+
(In reply to comment #6) > (From update of attachment 142643 [details] [diff] [review]) > The internal name for the system pref panel is winhooks :-) I do wonder why > this uses 1. and 2. instead of <ol> and <li> though... > Neil, can you check this in? This is just a simple correction of the fix in bug 210335, so it would be cool to have it in for the 1.7 release :) The lack of <ol> and <li> will be fixed in bug 236942.
(In reply to comment #7) >it would be cool to have it in for the 1.7 release :) Then you'd better request approval...
Attachment #142643 - Flags: approval1.7?
Comment on attachment 142643 [details] [diff] [review] Patch a=mkaply Please note this l10n change in seamonkey and l10n newsgroups and add the late-l10n keyword. Thanks
Attachment #142643 - Flags: approval1.7? → approval1.7+
Keywords: late-l10n
Patch checked in by Neil --> Fixed.
Status: ASSIGNED → RESOLVED
Closed: 21 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: