Timea Zsoldos [:zstimi/tzsoldos], Desktop QA Reporter
	Description • 6 years ago

[Affected versions]:
- Firefox 61
- Firefox 62
- Firefox 63
- Firefox 64.0b5
- Firefox nightly 65.0a1

[Affected platforms]:
- Windows 10 x64
- Mac OS X 10.13
- Ubuntu 16.04 x64

[Steps to reproduce]:
1. Download and start Firefox on an RTL arabic build from here: https://archive.mozilla.org/pub/firefox/candidates/64.0b5-candidates/build1/win64/ar/ 
2. Access the about:support page

[Expected result]:
- Everything on this page is align from right to left.

[Actual result]:
- The first column is align from left to right and the second from right to left 
(see: RTLaboutsupport.png)

[Additional notes]:
- I've managed to reproduce this with RTL builds like: "ar", "fa" or "he". 
- Also this issue is not reproducible from Fx55 to Fx60 and Fx60.3.0esr unaffected.

	:Gijs (he/him)
	Comment 1 • 6 years ago

This is a result of this CSS rule:

https://searchfox.org/mozilla-central/rev/eac6295c397133b7346822ad31867197e30d7e94/toolkit/themes/shared/in-content/info-pages.inc.css#162

which explicitly suggests that this is so file-paths are displayed LTR. Because of how mixed LTR/RTL works and has given us issues with URLs, I could imagine that just removing or overriding this rule here would cause problems because file paths contain mixed-directionality characters (like dir separators). A lot of the content in the screenshot is Latin text, and effectively en-US, so I don't think this is necessarily wrong...

I don't know that a lot needs to change here, and if it does, what exactly we should change. Itiel, can you maybe expand on how you think about:support should look in RTL, and what we could change here to make the experience better for RTL users?

	Itiel Assignee
	Comment 2 • 6 years ago

Thanks for asking!
This reminds me of bug 1329433.

I may be asking too much, but this is what I think should be ideal for RTL in this page:
1. Wherever non-latin characters are not expected, hardcode (the text only) to LTR. That's including file paths, which is an exception. But they all should still be aligned to the right.
2. All table headers (in blue) should be aligned to the right (if not centered). That's the current behaviour.
3. Localized RTL text should of course be RTL, by all means.
That should cover all text in about:support I think.

I'd rather have the Important Modified Preferences' actual preferences in LTR as these are sorted alphabetically and I think the human eye will be able to better locate any individual if they'd be in LTR, but as the 'value' column contents then would also have to be LTR, this will just look wrong. So I dropped this idea.

	Timea Zsoldos [:zstimi/tzsoldos], Desktop QA Reporter
	Comment 3 • 6 years ago

I managed to found a regression range manually: 
               Last Good: 20180402220122 
               First Bad: 20180403220040   
Pushlog: https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=445255800255bb13ed096b5b7da36aa835e41dd8&tochange=00bdc9451be6557ccce1492b9b966d4435615380  

Maybe this bug caused the issue: https://bugzilla.mozilla.org/show_bug.cgi?id=1373921

	Jared Wein [:jaws] (please needinfo? me)
	Comment 7 • 5 years ago

I didn't read in to the bug enough and now see that this is unrelated to the Fluent conversion. I don't have time to work on this right now.

	Pulsebot
	Comment 9 • 5 years ago

Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/c9dc0db9e9a5
Align the appropriate about:support table cells to the right for RTL r=Gijs