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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

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.
Blocks: 1035553
Assignee: nobody → jdarcangelo
Attached file pull-request (master)
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)
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)
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)
Hi Justin, 
would you mind rebasing your patch with the latest changes on master?
Best regards
(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 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+
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.)
Landed on master:

https://github.com/mozilla-b2g/gaia/commit/213352511efa6d41fbfb0b67b023a7d02b428ac4
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
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!
(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.