Closed Bug 1118049 Opened 5 years ago Closed 5 years ago
Can't enter 0
.1MB as value for data usage limit in the cost control app
This is reproducable with: Gaia-Rev 698e6e8a098cc060b26cd6f25171633c4c7e739d Gecko-Rev https://hg.mozilla.org/mozilla-central/rev/13fe5ad0364d Build-ID 20150102010205 Version 37.0a1 Device-Name flame FW-Release 4.4.2 FW-Incremental 39 FW-Date Thu Oct 16 18:19:14 CST 2014 Bootloader L1TC00011880 I think this used to work with a build from a week ago or so. Steps to reproduce: - Start up Usage app - Tap on the Settings icon - Tap on select just under "When use is above" -> Data usage limit dialog opens - Try to enter "0.1" as value in the input at the bottom Expected result: - 0.1 is entered as value Actual result: - 0.1 is not accepted as value This prevents bug 1115180 to be re-enabled.
Summary: Can't enter → Can't enter 0.1MB as value for data usage limit in the cost control app
Marina, Vivien, can you please take a look over this issue? We cannont re-enable a smoketest because of this. Thanks!
Assignee: nobody → marina.rodrigueziglesias
Hi Salva, would you mind reviewing the patch? Regards
Comment on attachment 8548692 [details] [review] patch v1.0 Not sure. We should check with UX if unit fractions are valid values. Delaying review until feedback from UX is provided. Juwei, could you please tell us if 0.1M or 0.1G are valid values for data alerts?
Hi, 0.1 makes sense to me. Thanks.
Comment on attachment 8548692 [details] [review] patch v1.0 Marina, address the comments on GitHub and ask for my review again. Thank you!
Comment on attachment 8548692 [details] [review] patch v1.0 Updated the pr with your comments
Comment on attachment 8548692 [details] [review] patch v1.0 Just a little comment on GitHub. Thank you!
Attachment #8548692 - Flags: review?(salva) → review+
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Comment on attachment 8548692 [details] [review] patch v1.0 [Approval Request Comment] [Bug caused by] (feature/regressing bug #): 1089829 [User impact] if declined: The user cannot introduce values that starts with 0. or . [Testing completed]: Yes [Risk to taking this patch] (and alternatives if risky): Low risk [String changes made]: No
Attachment #8548692 - Flags: approval-gaia-v2.2?(release-mgmt)
Attachment #8548692 - Flags: approval-gaia-v2.2?(release-mgmt) → approval-gaia-v2.2+
I just verified and this issue is fixed in latest mozilla-central. I am able to insert '0.1 MB' as mobile data limit alert. Gaia-Rev ebb5bd74ac660092d863c86e4ccf731b903be2e4 Gecko-Rev https://hg.mozilla.org/integration/b2g-inbound/rev/cccb55b1561d Build-ID 20150116052200 Version 38.0a1 Device-Name flame FW-Release 4.4.2 FW-Incremental eng.cltbld.20150116.085803 FW-Date Fri Jan 16 08:58:13 EST 2015 Bootloader L1TC10011880
Per comment 10, this bug is fixed/verified, so change status and "Tracking Flags" to "verified
You need to log in before you can comment on or make changes to this bug.