Closed Bug 1136466 Opened 6 years ago Closed 6 years ago

[RTL] Media player controls progress bar should NOT be mirrored if the rest of controls are not

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-v2.2 affected, b2g-master affected)

RESOLVED INVALID
Tracking Status
b2g-v2.2 --- affected
b2g-master --- affected

People

(Reporter: pcheng, Assigned: GaryChen)

References

Details

Attachments

(2 files)

RTL documentation states player controls progress bar should be mirrored, but after actually using those apps I found that it makes utilizing their controls buttons which are not mirrored counterintuitive. This issue affects the Music app, Video app, Gallery app, and Camera preview app (anywhere that playing a media is supported).

Prerequisites:
- In RTL language
- Have at least a music file and a video file available for playing

STR:
1) Go to Video app and start playing a video
2) Attempt to Fastforward and Rewind by tapping on those buttons
3) Go to Music app and attempt the same as step 2

Expected: Progress bar should NOT be mirrored. Fastforward button should be the button pointing to the direction of progress bar's movement when playing the media file, and vice versa for Rewind button.

Actual: Progress bar is mirrored. The opposite of intuitive behavior. Progress bar plays from right to left, but buttons contradict with it. See video for issue demonstration:

https://www.youtube.com/watch?v=DwSxB_hl7cA

Device: Flame 3.0 Master (full flash 319MB)
BuildID: 20150224010314
Gaia: 31ac1cd7a029d5e46dd7c92537b5c973c5d9826e
Gecko: 368c62292249
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 39.0a1 (3.0 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
This issue also occurs on Flame 2.2. Progress bar is mirrored.

Device: Flame 2.2
BuildID: 20150224002637
Gaia: 8e98fe665f3821d10d4d982cbb14cbe5b94d0be5
Gecko: 2b70d9d62799
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0 (2.2) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [rtl-impact][QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Assignee: nobody → gchen
Component: Gaia → Gaia::Music
Attached file pr for master
Hi Dominic,
   Could you help to review this patch?
   Thanks.
Attachment #8569043 - Flags: review?(dkuo)
QA Whiteboard: [rtl-impact][QAnalyst-Triage?] → [rtl-impact][QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Invalid, this screen is very clearly called out in the spec and has been vetted with native RTL users. Progress is expected to move from right to left and due to hardware legacies it is understood by users that the next/previous buttons do not match directionality of the progress bar.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
Comment on attachment 8569043 [details] [review]
pr for master

clean review flag since set as invalid.
Attachment #8569043 - Flags: review?(dkuo)
You need to log in before you can comment on or make changes to this bug.