350594 - Horizscroll should be negated for RTL locales

Status: NEW

(Core :: Layout: Text and Fonts, enhancement)

Description

[Behnam Esfahbod [:zwnj]]

Values of mousewheel.horizscroll.*.numlines should be negated in RTL locales, as up/down, back/forward, etc are reverse.

Currently, I set them on "fa" firefox-l10n.js, but Pike needs approval for this.

Comment 1

[Behnam Esfahbod [:zwnj]]

RTL locales really need it.

IMHO the best way for the branch is to put them on firefox-l10n.js.  Pike, may I put them for fa/Persian?

Comment 2

[Axel Hecht [:Pike]]

I'm not convinced. I tried to get this working with my mouse, and I didn't get it to work at all, which may be the fault of my mouse driver.

Given that this is really minor, this doesn't block a release, nor do I want to take changes to firefox-l10n.js without verifying what this does in real life.

I'd need to understand if this is really a locale specific thing (I doubt it right now) or something that depends on RTL on a textinput field, and, even more so, if my guess on that being only relevant in textinput fields is right.

Unsetting milestone, lowering severity. Not going to take that on Firefox 2, likely not code fixes, nor hacks around those.

Comment 3

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).

Comment 4

[(no longer active)]

Behnam, can you please provide more information on what kind of problem this produces, and whether or not it's present on trunk/1.9.1?  Thanks!