1102334 - AM/PM abbreviations are displayed on the wrong side of times in the call log

Status: RESOLVED INVALID

(Firefox OS Graveyard :: Gaia::Dialer, defect)

Description

[Doug Sherk (:drs) (inactive)]

Attachment: Screenshot

I can't find any conclusive information about this, but it seems like the AM/PM abbreviations are displayed on the wrong side of the time. My only source is the BiDi scoping spec, page 11, screen 4, label I.

Comment 1

[Doug Sherk (:drs) (inactive)]

Stephany, could you provide an assessment here?

ccing Oleg and Julien as the Messages app has the same potential problem.

Comment 2

[Julien Wajsberg [:julienw]]

The spec is https://mozilla.app.box.com/s/0y1amh4rwpp6brcxd1hk

IMO the spec looks weird and what we have looks better. NI Ahmed too.

Comment 3

[Ahmed Nefzaoui [:Nefzaoui]]

Actually the current position of AM/PM in attachment 8526101 [details] is the correct one.
RTL is all about reading from the right, and that applies to time too (basically reading time digits from the left while the whole composition of TIME + AM/PM is read from the right)
Another great example is the digital clock in Clock app :)

Comment 4

[Doug Sherk (:drs) (inactive)]

Resolving based on comment 3.

Comment 5

[Stephany Wilkes]

Many people, from various Arabic regions, reviewed the pattern document before it was finalized. If we're changing anything in that document for any reason, we need to know why, take it to Frameworks, and document that. 

Ahmed, please let me know if you're suggesting a pattern change so we can send out the suggestion for review. Thanks!

Comment 6

[Ahmed Nefzaoui [:Nefzaoui]]

This particular case doesn't exists yet in our spec so not a suggestion to change but rather suggesting an addition to the spec, which is the explanation in comment 3 on how time and the AM/PM labels are ordered :)
The best correct example is the digital clock in Clock app

Comment 7

[Julien Wajsberg [:julienw]]

Stephany, I guess the only thing to be changed in the current spec is, as Doug said, the BiDi scoping spec, page 11, screen 4, label I.