-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use Arabic numerals consistently #17002
Comments
I've investigated the issue and for me the behaviour is a bit different. All numbers on the stats screen are displayed in Indo-Arabic for me: From this StackOverflow answer I understood that there was an issue filed for the Android code. Unfortunately the link to the issue doesn't work for me, saying I'm denied access (tried with 1 e-mail addresses), so I couldn't check what's going on behind the scenes and what is the latest status update on it. While continuing to investigate the issue I've discovered another strange thing, related to the time formatting for the "best hour". When changing the app language to Arabic and looking at the stats → insights tab, the hour is displayed and formatted with Indo-Arabic digits, as seen in the screenshot above. If I close the app and open it again (thus the initial locale on app start is Arabic), the hour is formatted with Western Arabic numbers, and the AM/PM is shown in English: I've started working on a fix for the original issue, but I won't be addressing this newly discovered problem in that fix, since it proves to be a different issue, which probably has its root cause in the localization system we're using. There's currently progress on updating that. |
This PR fixes the issue for Stats: #17160 We can apply the same approach introduced in that PR to all other places where we're displaying numerals on the UI, the solution in that PR is easily applicable in most cases; it should be fairly simple to reuse it. The task of finding all such places is quite complex though :) |
@ovitrif thanks so much for working on this!
Yeah, that seems quite likely. I think our primary concern should be to make sure everything looks as it should using the device language settings.
Fair point! We could do a physical device audit to look for these numerals on all screens. I wonder though, is there a class or type based strategy we could use to find all such places in the codebase? |
I agree, thank you @thehenrybyrd 🙇 !
That would be great indeed but it might not work in this case according to my current knowledge of the issue. We could in principle replace all usages of text
In all fairness my current understanding tells me this might be the best approach to fix the issue in all places. There's definitely a big amount of work required for this, but with this approach we're sure to limit the changes to what's needed 👍 |
@ovitrif that makes sense to me! This isn't high enough priority for an audit currently, but I'll keep reporting other instances when I run across them. Thanks for fixing this for stats! |
Upon beta-testing the fixes for the in-app Stats, we've found another place where Eastern Arabic numerals should be replaced with Western Arabic ones:
|
Expected behavior
Arabic numerals (1 2 3 4 5 6 7 8 9 0) should be used everywhere all the time.
Actual behavior
When Arabic is the selected app language, in a number of places, but not consistently, we use Indo-Arabic numerals instead of normal Arabic numerals. We would do better to consistently use Arabic numerals (the regular ones we’re used to) rather than Eastern Arabic (Indo-Arabic) numerals – (٠ ١ ٢ ٣ ٤ ٥ ٦ ٧ ٨ ٩) – because everyone who reads Arabic knows the Arabic numerals, but not all know the Indo-Arabic numerals.
This looks especially strange on screens where we use both types of numerals:
Steps to reproduce the behavior
I've seen them in the Stats Card and Screens, Blogging Prompts Card, while selecting Media, and in Reader (dates)
Tested on Samsung Galaxy S21, Android 12, WPAndroid 20.4-rc-2
Internal reference: p5T066-3jv-p2#comment-12698
The text was updated successfully, but these errors were encountered: