Closed
Bug 1093563
Opened 10 years ago
Closed 10 years ago
[Flame][v2.1&v2.0][Music]Music time bar operate abnormal when change right-to-left language.
Categories
(Firefox OS Graveyard :: Gaia::Music, defect)
Tracking
(b2g-v2.2 verified)
VERIFIED
FIXED
2.1 S9 (21Nov)
Tracking | Status | |
---|---|---|
b2g-v2.2 | --- | verified |
People
(Reporter: whsu, Assigned: mancas)
References
Details
Attachments
(3 files)
+++ This bug was initially created as a clone of Bug #1091455 +++
This a functional defect and reported by our partner.
Please fix it on v2.1.
* Precondition:
- Select first language in language list
- Have some songs in phone.
* Steps to reproduce:
1) Main menu->Play music->Select one song to play->Tap time bar
2) Check the time bar display.
* Actual results
- Time bar operate abnormal when change right-to-left language.
(Demo video: WP_20141104_009.mp4)
* Expected results:
- Time bar should operate normally when change right-to-left language.
* Build Information
@ Tako
Gaia-Rev d056733f8a7a1a152f5458b323f092c47dbffa48
Gecko-Rev df0479fb8c18379488013452e903c33bca384b0d
Build ID 20141024145043
Version 32.0
@ Flame
Gaia-Rev 8b0cf889ae0d48a9eb7ecdcb9b67590de45cc5e5
Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/388b03efe92d
Build-ID 20141104001202
Version 34.0
* Reproduce Rate ==
100%
Reporter | ||
Comment 2•10 years ago
|
||
[Blocking Requested - why for this release]:
This a functional defect and reported by our partner.
Please fix it on v2.1. Thanks.
Reporter | ||
Comment 3•10 years ago
|
||
[Blocking Requested - why for this release]:
blocking-b2g: --- → 2.1?
Comment 4•10 years ago
|
||
We are looking into all RTL related issues across all apps in 2.2. Adding this to the music-rtl meta bug.
Thanks
Hema
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → b.mcb
Assignee | ||
Comment 5•10 years ago
|
||
Hey David! Could you review the patch, please?
Let me know if we need to make some changes in the patch.
Thanks!
Attachment #8517370 -
Flags: review?(dflanagan)
Comment 7•10 years ago
|
||
Comment on attachment 8517370 [details]
Proposed patch
Hub,
Since you're handling RTL compliance for the Music app, I'm passing this review on to you. If you don't think you're familiar enough with that part of the code to review, feel free to send it back to me.
Attachment #8517370 -
Flags: review?(dflanagan) → review?(hub)
Comment 8•10 years ago
|
||
Comment on attachment 8517370 [details]
Proposed patch
Looks good to me.
Attachment #8517370 -
Flags: review?(hub) → review+
Assignee | ||
Updated•10 years ago
|
Keywords: checkin-needed
Comment 9•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
status-b2g-v2.2:
--- → fixed
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 2.1 S9 (21Nov)
Comment 11•10 years ago
|
||
This issue verified successfully on Flame 2.2:
Gaia-Rev 7c5b27cad370db377b18a742d3f3fdb0070e899f
Gecko-Rev https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/ce27f2692382
Build-ID 20150115002505
Version 37.0a2
Reproduce rate 0/5
Comment 12•10 years ago
|
||
Test case has been added in moztrap:
https://moztrap.mozilla.org/manage/case/15094/
You need to log in
before you can comment on or make changes to this bug.
Description
•