Closed Bug 947028 Opened 6 years ago Closed 6 years ago

[Buri][Settings] System and Purchased Ringtones overlap check boxes

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set

Tracking

(blocking-b2g:1.3+, b2g-v1.3 fixed)

RESOLVED FIXED
blocking-b2g 1.3+
Tracking Status
b2g-v1.3 --- fixed

People

(Reporter: marcia, Assigned: pdahiya)

References

Details

(Keywords: regression)

Attachments

(2 files)

Buri, while running:

Gaia   1dd0e5c644b4c677a4e8fa02e50d52136db489d9
SourceStamp 725c36b5de1a
BuildID 20131205040201
Version 28.0a1

STR:
1. Settings | Sound | Ringer
2. Open the menu 
3. Observe the attached screenshot

This works correctly using a 1.2 build:

Gaia   0659f16b9790b1cf9eba4d80743fcc774d2ffe3a
SourceStamp af2c7ebb5967
BuildID 20131205004003
Version 26.0
Keywords: qawanted
This might be another fallout from the screen reader work. A regression window will confirm that.
Keywords: qawanted
QA Contact: nkhristoforov
The regression window for this bug is 11-13 to 11-14.

Last Working Build:
Device: Buri v1.3 Moz RIL
BuildID: 20131113040200
Gaia: 7243b75041c93bc7deb240131748d0b164f3f0b0
Gecko: 7b014f0f3b03
Version: 28.0a1

First Broken Build:
Device: Buri v1.3 Moz RIL
BuildID: 20131114143149
Gaia: 4ea5adac92d6a9ab035e708e941d50be0d670da2
Gecko: 7b014f0f3b03
Version: 28.0a1
Yup, it's another fallout from bug 914892.
Blocks: 914892
Duplicate of this bug: 947472
Assignee: nobody → pdahiya
Triage: regression.
blocking-b2g: 1.3? → 1.3+
Hi David, Attached is the fix inside ringtones.css for the overlapping option buttons. Please review. Thanks
Attachment #8345382 - Flags: review?(dflanagan)
Comment on attachment 8345382 [details] [review]
Pull request with the fix of overlapping check boxes

Trivial fix. Looks good to me.
Attachment #8345382 - Flags: review?(dflanagan) → review+
On second thought...  The subject line of this bug implies that purchased ringtones have the same bug. Has anyone actually checked that? The attached patch does not fix purchased ringtones, so we may well need to make the same change in apps/fl/style/pick.css
(In reply to David Flanagan [:djf] from comment #9)
> On second thought...  The subject line of this bug implies that purchased
> ringtones have the same bug. Has anyone actually checked that? The attached
> patch does not fix purchased ringtones, so we may well need to make the same
> change in apps/fl/style/pick.css

Good catch, i have fixed and tested UI for purchased ringtones. Please review. Thanks
Flags: needinfo?(dflanagan)
Synched up with djf over IRC and the fix looks good. Carrying over the r+ and landing the patch on master
Flags: needinfo?(dflanagan)
Patch merged with master https://github.com/mozilla-b2g/gaia/commit/dea74850442df6aff8405333a973d49a9f1af2ec
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Duplicate of this bug: 949718
Duplicate of this bug: 956981
Uplifted dea74850442df6aff8405333a973d49a9f1af2ec to:
v1.3: 07315d8a283fae4fb24e9f3cc65532ec933f255b
Still seeing this on Buri 1.3 mozilla RIL.  However, it does not appear on 1.4.  Should I write a new bug?

Buri 1.3
Build ID: 20140102004001
Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/61f553e5db49
Gaia: 01e9da49be2cc4bc134eeefc434740d572ec2246
Platform Version: 28.0a2
Firmware Version: V1.2-device.cfg
(In reply to ckreinbring from comment #16)
> Still seeing this on Buri 1.3 mozilla RIL.  However, it does not appear on
> 1.4.  Should I write a new bug?
> 
> Buri 1.3
> Build ID: 20140102004001
> Gecko: http://hg.mozilla.org/releases/mozilla-aurora/rev/61f553e5db49
> Gaia: 01e9da49be2cc4bc134eeefc434740d572ec2246
> Platform Version: 28.0a2
> Firmware Version: V1.2-device.cfg

Yes - file a new bug.
You need to log in before you can comment on or make changes to this bug.