1106170 - Call direction indicators are pointed the wrong way in RTL

Status: RESOLVED INVALID

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

Description

[Doug Sherk (:drs) (inactive)]

Attachment: Screenshot

The direction indicator for calls on the Callscreen are pointed the wrong way. This is also a problem in the call log, and anywhere else we use the direction indicators.

Comment 1

[Doug Sherk (:drs) (inactive)]

Ahmed, am I correct here that this is an issue, or are the current visuals what you would expect?

Stephany, is there a plan to deal with this? We have several potential solutions here that I can think of:
1. (my preferred) Create a new set of icons for RTL that are identical but mirrored.
2. Mirror the icons dynamically. Not sure about the perf implications here. It also artificially limits us to being unable to do anything like just reversing the direction of the arrow, instead of the phone receiver part of the icon as well.
3. Leave them as-is.

Comment 2

[Stephany Wilkes]

Hi Doug - Per the bidi pattern (at https://mozilla.box.com/s/0y1amh4rwpp6brcxd1hk), we are not changing iconography for this release. We may do it later if we have more time. During pattern review, icons were not found to communicate time-ordered direction strongly or a critical issue to fix.

Comment 3

[Ahmed Nefzaoui [:Nefzaoui]]

For the records, I'd go with solution 1 too, a set of RTL icons that are mirrored when we're using icon-font, but in case of PNG, just mirror and compress the asset. :)
Closing based on Comment 2.

Comment 4

[Dylan Oliver [:doliver]]

RTL triage: P2 -- will make a best effort to get this into the 2.2 release.

Comment 5

[Dylan Oliver [:doliver]]

RTL update: marking required bugs as feature-b2g:2.2+ (and removing blocking flags)

Comment 6

[Doug Sherk (:drs) (inactive)]

Per comment 2 and comment 3, I'm closing this for now. If we decide to update our iconography, we can reopen this or open another bug with more details.

Comment 7

[Alissa（leave from Mozilla）]

Test case has been added in moztrap:
https://moztrap.mozilla.org/manage/case/15425/