233647 - "Mozilla Keyboard Shortcuts" doesn't cover Solaris

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Leon Sha]

In the Mozilla online help in Mozilla 1.4 beta for Solaris, the Mozilla 
Keyboard Shortcuts section has charts listing the keyboard shortcuts for
Windows, Macintosh, and Linux, but not Solaris.
Suggest to modify Linux to Linux/Unix

Comment 1

[Leon Sha]

Attachment: Patch for help

Comment 2

[Leon Sha]

Comment on attachment 141033 [details] [diff] [review]
Patch for help

Neil, can you help to review this patch.

Comment 3

[[e-mail]]

Comment on attachment 141033 [details] [diff] [review]
Patch for help

I think these should be rewritten to look like a list i.e. "(Windows, Linux,
Unix)" or "Mac OS, Linux or Unix".
Also,I actually prefer to use "or" to connect the alternatives, rather than
"and" which is currently used in some cases, but you should double-check with
rj_keller.

Comment 4

[R.J. Keller]

> I think these should be rewritten to look like a list i.e. "(Windows, Linux,
> Unix)" or "Mac OS, Linux or Unix".

I assume you mean the Keyboard shortcuts. What do you mean like a list? You mean
going from Windows down to Linux for all the shortcuts? I wouldn't like that
format too much.

> Also,I actually prefer to use "or" to connect the alternatives, rather than
> "and" which is currently used in some cases

What do you mean by "or", Neil? You mean using "or" instead of a slash?

Comment 5

[[e-mail]]

The slash for both - I just don't like seeing Linux/Unix everywhere.
Instead of "Windows and Linux/Unix" I prefer "Windows, Linux or Unix".

Comment 6

[Leon Sha]

(In reply to comment #4)
> > I think these should be rewritten to look like a list i.e. "(Windows, Linux,
> > Unix)" or "Mac OS, Linux or Unix".
> 
> I assume you mean the Keyboard shortcuts. What do you mean like a list? You mean
> going from Windows down to Linux for all the shortcuts? I wouldn't like that
> format too much.

I think neil want to list the shortcuts for unix with a seperate column.

Comment 7

[Leon Sha]

Attachment: Change linux/unix to linux or unix

New patch with neil's comments.

Comment 8

[[e-mail]]

Comment on attachment 141295 [details] [diff] [review]
Change linux/unix to linux or unix

Almost there...

>-Mac OS and Linux. </p>
>+Mac OS and Linux or Unix. </p>
These should say "Mac OS, Linux or Unix." The same idea would apply to
"Windows, Linux or Unix."

>+<p><strong>Note:</strong> This feature is not available on Linux or Unix &amp; Mac.</p>
rj_keller, I noticed we're not consistent here, sometimes we use Macintosh OS,
sometimes Mac OS, and this one just says Mac. Should we stick to Mac OS?
leon.sha, in that case this would say "Mac OS, Linux or Unix" too.

Comment 9

[R.J. Keller]

> rj_keller, I noticed we're not consistent here, sometimes we use Macintosh OS,
> sometimes Mac OS, and this one just says Mac. Should we stick to Mac OS?

We should use Mac OS because Mac seems to refer to the platform more than the OS
(which is what we're listing).

There's a lack of consistency in many areas of the Help Viewer.

Comment 10

[Leon Sha]

Attachment: Patch with neil's comments

Comment 11

[[e-mail]]

Comment on attachment 141494 [details] [diff] [review]
Patch with neil's comments

> <p><b>Note</b>: Some Page Setup functions are different or unavailable on Mac OS
>-and Linux. </p>
>+, Linux or Unix. </p>
Just one tiny nit (well, it happens in two files) - this displays as:
Mac OS , Linux or Unix
Please can you change these lines to:
<p><b>Note</b>: Some Page Setup functions are different or unavailable on
Mac OS, Linux or Unix. </p>

Comment 12

[Leon Sha]

Attachment: Patch with neil's comments.

Neil, can you tell me who is the sr of this module?

Comment 13

[[e-mail]]

Help text doesn't need sr; you can just check it in when the tree reopens.

Comment 14

[R.J. Keller]

Comment on attachment 141523 [details] [diff] [review]
Patch with neil's comments.

Leon, I don't require Help Text to be super-reviewed unless it was a major
layout change of some sort. XUL and JS changes would be done by the same sr's
as the XPApps module (alecf or jag would probably be best).

r=[e-mail], let's try and get 1.7a approval for this (it's small, so why
not?).

Comment 15

[R.J. Keller]

--> Help, for flags.

Comment 16

[chris hofmann]

Comment on attachment 141523 [details] [diff] [review]
Patch with neil's comments.

a=chofmann for 1.7a

Comment 17

[R.J. Keller]

Fix checked in.

Thanks for the patch, Leon!

Comment 18

[Stefan [:stefanh]]

Reopening since the current composer_help file lacks changes made after
revision 1.14. This will be fixed by the patch in bug 210335.

Comment 19

[Stefan [:stefanh]]

--> Fixed