1119943 - [RTL][Email] Sync icon is not mirrored

Status: RESOLVED INVALID

(Firefox OS Graveyard :: Gaia::E-Mail, defect, P3)

Description

[Yeojin Chung [:YeojinC]]

Description:
The Continuous/spinner icon for the sync is not mirrored, and the animation also does not flow counter clockwise.

Pre-requisite: Set up an email account in Email app.

Repro Steps:
1) Update a Flame device to BuildID: 20150109010206.
2) Set the device language in Arabic under Settings > Language.
3) Open Email app.
4) Select the sync icon.

Actual:
The sync icon is not mirrored, and the animation flows clockwise.
  
Expected: 
The sync icon is mirrored, and the animation flows counter-clockwise.
  
Environmental Variables:
Device: Flame 2.2 Master (319mb)(Kitkat Base)(Full Flash)
BuildID: 20150109010206
Gaia: 2c7d14040149e1f9b1bb3972ff150be0472fa6b6
Gecko: b3f84cf78dc2
Gonk: a814b2e2dfdda7140cb3a357617dc4fbb1435e76
Version: 37.0a1 (2.2 Master)
Firmware: V188-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

Repro frequency: 100%
See attached: video

Comment 1

[James Burke [:jrburke]]

It was my understanding from previous RTL guidance/specs that we were not changing the direction of animations for RTL support. Are there new guidelines since Nov 2014? Also asking swilkes since she gave the original guidance to us.

Comment 2

[Dylan Oliver [:doliver]]

The arrow in the sync icon was not deemed to be directional in a meaningful sense, so we have no plans to mirror it. In other words, whether it is presented in clockwise or counter-clockwise, that does nothing to help or hinder the meaning of the symbol.

Comment 3

[Yeojin Chung [:YeojinC]]

I filed a bug based on p.16 on the spec sheet (https://mozilla.app.box.com/s/0y1amh4rwpp6brcxd1hk), which shows the exact issue on the email app. I guess the document has to be updated now. Thanks for the clarification.

Comment 4

[Delphine Lebédel [:delphine - Use needinfo]]

@James: yes, there are updated guidelines that Stephany sent us. You can find them here: https://mozilla.app.box.com/bidi-22/1/2653676065/22656349421/1
From what I see, this should be mirrored (page 15)
Thus re-opening

Comment 5

[Delphine Lebédel [:delphine - Use needinfo]]

sorry, meant page 16 of the specs

Comment 6

[Stephany Wilkes]

Everyone is correct: this is a recent change we made to the "living," always changing spec based on feedback received from the Mozilla Arabic community during our Portland work week. This change is, however, not critical for any particular release. 

I updated the pattern just a few days ago to reflect this change so, at this date, there has not been time to make this change. I would mark this new icon as something we would want post 2.2, based on the date of the changes to the pattern.

Comment 7

[Stephany Wilkes]

Delphine, please make sure QA for this release is testing based on the November 2014 (v4 I think) version of the pattern of the spec, and not the very recently changed January spec. There will have been no time to have made the changes to the January spec.

Comment 8

[James Burke [:jrburke]]

swilkes: it would be good to know if there is an update/change notification process for that spec, so that I can get on it. The changes were new to me, and want to be sure I am up to date if I need to do triage or bug review. If it can be public, sending a note to dev-gaia when there are new revisions may be all that is needed.

Comment 9

[Dylan Oliver [:doliver]]

RTL triage: P3 -- this bug is not prioritized for inclusion in the 2.2 release.

Comment 10

[Norry.L.F(Leave from Mozilla)]

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

Comment 11

[Delphine Lebédel [:delphine - Use needinfo]]

Closing this bug again as for 2.2, the Nov. specs indicated that we are *not* changing the direction of animations. 
Norry please make sure test cases reflect that. thanks.