Closed
Bug 1077546
Opened 8 years ago
Closed 8 years ago
[Cost Control] Add app detail view
Categories
(Firefox OS Graveyard :: Gaia::Cost Control, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: justindarc, Assigned: justindarc)
References
Details
Attachments
(1 file)
As part of the user story in Bug 1035553, tapping on an app in the list on the main screen should take the user to an "app detail" view. From the app detail view, the user should be able to view usage specific to that one app. Bug 1035553 will then simply track the addition of a toggle switch to the app detail view to enable/disable cellular data.
Assignee | ||
Updated•8 years ago
|
Assignee: nobody → jdarcangelo
Assignee | ||
Comment 1•8 years ago
|
||
Marina: I have moved the patch from Bug 1035553 here with the exception of the temporary removal of the toggle switch in the app detail view.
Attachment #8499687 -
Flags: review?(mri)
Assignee | ||
Comment 2•8 years ago
|
||
Comment on attachment 8499687 [details] [review] pull-request (master) Also adding salva for review on this as well as this patch has been ready for review for over a week now.
Attachment #8499687 -
Flags: review?(salva)
Assignee | ||
Comment 3•8 years ago
|
||
Comment on attachment 8499687 [details] [review] pull-request (master) Removing salva from review. Spoke to Marina and we will hold off on reviewing/landing this until some remaining v2.1 patches have been landed and uplifted (to prevent having to rebase since this is a fairly large patch).
Attachment #8499687 -
Flags: review?(salva)
Comment 4•8 years ago
|
||
Hi Justin, would you mind rebasing your patch with the latest changes on master? Best regards
Assignee | ||
Comment 5•8 years ago
|
||
(In reply to Marina Rodríguez [:mai] from comment #4) > Hi Justin, > would you mind rebasing your patch with the latest changes on master? > Best regards Marina: Done!
Flags: needinfo?(mri)
Comment 6•8 years ago
|
||
Comment on attachment 8499687 [details] [review] pull-request (master) Nice work. You have a little comment on the PR. Thank you!
Flags: needinfo?(mri)
Attachment #8499687 -
Flags: review?(mri) → review+
Comment 7•8 years ago
|
||
Nowadays, exists a patch to fix the difference between by application breakdown totals and the total by interface (residual traffic). After landing the patch, this difference will be reflected on the system app detail view. This issue must be resolved when the residual traffic was 0. (Bug 1083680 - [Usage][Cost Control] Noticeable difference between by application breakdown totals and the total displayed in chart and widget.)
Assignee | ||
Comment 8•8 years ago
|
||
Landed on master: https://github.com/mozilla-b2g/gaia/commit/213352511efa6d41fbfb0b67b023a7d02b428ac4
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Comment 9•8 years ago
|
||
The l10n part doesn't work, we need plurals here :/ Could you open a follow-up to add plural forms for data-used-since, data-used-this-week and data-used-this-month, please? (And update the ID since it has landed) You'll also need to use two variable instead, like: data-used-this-week = {{n}} {{unit}} of data used this week Thanks!
Assignee | ||
Comment 10•8 years ago
|
||
(In reply to Théo Chevalier [:tchevalier] from comment #9) > The l10n part doesn't work, we need plurals here :/ Could you open a > follow-up to add plural forms for data-used-since, data-used-this-week and > data-used-this-month, please? (And update the ID since it has landed) > > You'll also need to use two variable instead, like: > data-used-this-week = {{n}} {{unit}} of data used this week > > Thanks! Théo: Filed follow-up here: Bug 1088786 Thanks for pointing that out!
See Also: → 1088786
You need to log in
before you can comment on or make changes to this bug.
Description
•