823647 - Update Mouse Wheel preferences help after bug 782143 and bug 819652

Status: RESOLVED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Jens Hatlak (:InvisibleSmiley)]

Bug 819652 changed the Mouse Wheel preferences pane. Help needs to be adapted accordingly.

Comment 1

[[e-mail]]

819652? Did we ever update it after bug 782143? Oops...

Comment 2

[[e-mail]]

Attachment: Possible patch

Comment 3

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 694529 [details] [diff] [review]
Possible patch

Review of attachment 694529 [details] [diff] [review]:
-----------------------------------------------------------------

Too many issues for a positive review or feedback, but thank you very much for getting to this so fast! :-)

::: suite/locales/en-US/chrome/common/help/cs_nav_prefs_advanced.xhtml
@@ +548,5 @@
>  <ul>
>    <li>Select <q>No modifier key</q> or select a key that you want to use
>      along with the mouse wheel. Use the checkboxes below to configure mouse
>      wheel behavior:
>      

Nit: Remove whitespace while we're here (see Splinter Review).

@@ +550,5 @@
>      along with the mouse wheel. Use the checkboxes below to configure mouse
>      wheel behavior:
>      
>      <ul>
> +      <li><strong>Same as vertical</strong>: Choose this when your mouse has a

Should read "Same as vertical scrolling".

@@ +553,5 @@
>      <ul>
> +      <li><strong>Same as vertical</strong>: Choose this when your mouse has a
> +        single control that provides both vertical and horizontal scrolling.
> +        If your mouse has a separate horizontal scrolling control then you can
> +        choose to use assign a different function to it.</li>

Just "assign", I assume? ;-)

@@ +570,5 @@
>  
> +    <p>Use the <strong>Mouse wheel speed</strong> option to increase or
> +      decrease the responsiveness of the wheel. The <strong>Reverse
> +      direction</strong> option provides a convenient way of choosing a
> +      negative wheel speed.</p>

I wonder whether speed can be negative at all (at least in common speech; I'm even less sure about physics). I think it's wrong in this context. Maybe "reverse scrolling"?

Comment 4

[[e-mail]]

(In reply to Jens Hatlak from comment #3)
> (From update of attachment 694529 [details] [diff] [review])
> > +      <li><strong>Same as vertical</strong>: Choose this when your mouse has a
> Should read "Same as vertical scrolling".
Indeed.

> > +        If your mouse has a separate horizontal scrolling control then you can
> > +        choose to use assign a different function to it.</li>
> Just "assign", I assume? ;-)
Yeah, I changed my mind over the wording and didn't fix it properly.

> > +    <p>Use the <strong>Mouse wheel speed</strong> option to increase or
> > +      decrease the responsiveness of the wheel. The <strong>Reverse
> > +      direction</strong> option provides a convenient way of choosing a
> > +      negative wheel speed.</p>
> I wonder whether speed can be negative at all (at least in common speech;
> I'm even less sure about physics). I think it's wrong in this context. Maybe
> "reverse scrolling"?
I couldn't think of a better way of expressing this. How about:
Use the *Mouse wheel speed* option to increase, decrease or completely reverse the response of the wheel.

Comment 5

[Jens Hatlak (:InvisibleSmiley)]

(In reply to [e-mail] from comment #4)
> I couldn't think of a better way of expressing this. How about:
> Use the *Mouse wheel speed* option to increase, decrease or completely
> reverse the response of the wheel.

I like "responsiveness" better than "response" in connection with in/decrease. Also, the "completely" sounds too much like "totally" to me, i.e. doesn't really add much value in my eyes.

How about this:
"Use the *Mouse wheel speed* option to increase or decrease the responsiveness of the wheel. Use *Reverse direction* to change the scroll direction."

Comment 6

[[e-mail]]

(In reply to Jens Hatlak from comment #5)
> (In reply to comment #4)
> > I couldn't think of a better way of expressing this. How about:
> > Use the *Mouse wheel speed* option to increase, decrease or completely
> > reverse the response of the wheel.
> 
> I like "responsiveness" better than "response" in connection with
> in/decrease. Also, the "completely" sounds too much like "totally" to me,
> i.e. doesn't really add much value in my eyes.
> 
> How about this:
> "Use the *Mouse wheel speed* option to increase or decrease the
> responsiveness of the wheel. Use *Reverse direction* to change the scroll
> direction."

Well, the point is that *Reverse direction* works by negating the speed...

Comment 7

[[e-mail]]

I think I should let the help peers fight amongst themselves ;-)

Comment 8

[Stefan [:stefanh]]

OK, so you can either use negative numbers in "Mouse wheel speed" or select the "Reverse direction" checkbox which will then make the "Mouse wheel speed" numbers negative...

How about:
"Use the *Mouse wheel speed* option to adjust the response of the wheel. To reverse the scroll direction, use negative numbers or select *Reverse direction*."

Comment 9

[Jens Hatlak (:InvisibleSmiley)]

(In reply to [e-mail] from comment #6)
> Well, the point is that *Reverse direction* works by negating the speed...

Heh you're right, didn't check that! But that's mostly a technical aspect.

(In reply to Stefan [:stefanh] from comment #8)
> How about:
> "Use the *Mouse wheel speed* option to adjust the response of the wheel. To
> reverse the scroll direction, use negative numbers or select *Reverse
> direction*."

This has my vote, thanks! :-)

[I'll let the English native speakers decide whether "response" or "responsiveness" fits better in this context.]

Comment 10

[Ian Neal]

(In reply to Jens Hatlak (:InvisibleSmiley) from comment #9)
> (In reply to [e-mail] from comment #6)
> > Well, the point is that *Reverse direction* works by negating the speed...
> 
> Heh you're right, didn't check that! But that's mostly a technical aspect.
> 
> (In reply to Stefan [:stefanh] from comment #8)
> > How about:
> > "Use the *Mouse wheel speed* option to adjust the response of the wheel. To
> > reverse the scroll direction, use negative numbers or select *Reverse
> > direction*."
> 
> This has my vote, thanks! :-)
> 
> [I'll let the English native speakers decide whether "response" or
> "responsiveness" fits better in this context.]

My preference would be for responsiveness

Comment 11

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch v2 [Checkin: Comment 15]

I know this is mostly yours, but I don't speak in your name, so let's *reverse the direction* of the review situation. :-)

Comment 12

[[e-mail]]

Comment on attachment 695731 [details] [diff] [review]
patch v2 [Checkin: Comment 15]

I hope the response, if not the responsiveness, of the review is satisfactory ;-)

Comment 13

[Jens Hatlak (:InvisibleSmiley)]

Bah, sorry I didn't catch this earlier: the last of the radio button labels, formerly "Make the text larger or smaller", actually reads "Zoom the page in or out" now.

Neil, are you OK with just applying the label change to the help file, or do you think the explanation needs to be adjusted, too?

Comment 14

[[e-mail]]

(In reply to Jens Hatlak from comment #13)
> Bah, sorry I didn't catch this earlier: the last of the radio button labels,
> formerly "Make the text larger or smaller", actually reads "Zoom the page in
> or out" now.
> 
> Neil, are you OK with just applying the label change to the help file, or do
> you think the explanation needs to be adjusted, too?

Well, ideally you would link to the help for the appearance - content - zoom only text checkbox, but that might be beyond the scope of this bug, in which case I would just remove the words "text on".

Comment 15

[Jens Hatlak (:InvisibleSmiley)]

Comment on attachment 695731 [details] [diff] [review]
patch v2 [Checkin: Comment 15]

http://hg.mozilla.org/comm-central/rev/7e37758d13d9
with comments 13/14 addressed