840918 - [Cost Control]Current progress rate in progress bar will be displayed outside.

Status: RESOLVED FIXED

(Firefox OS Graveyard :: Gaia::Cost Control, defect)

Description

[Leo]

1. Title : Current progress rate in progress bar will be displayed outside.
2. Precondition : Set the data limit to 1GB and browse upto 3MB.
3. Tester's Action : Open cost control data widget and check the progress bar.                           
4. Detailed Symptom (ENG.) : Data usage rate is displayed outside the progress bar.(Same issue can be reproduced after data usage limit exceeds)
5. GAIA version : f46757775e07d314913b4a5ae975fa131d1c7c4b
6. Expected : Progress bar should show properly.
7.Reproducibility: Y
1)Frequency Rate : 100%

Comment 1

[Leo]

Attachment: Data widget UI break

Comment 2

[Salvador de la Puente González [:salva]]

Asking UX for prioritization.

Comment 3

[marcoc]

Should be a UX-P1. 
It seems like the basic functionality of the data widget graph is not working.

Comment 4

[marcoc]

Got some more info about this as I was having difficulty re-creating the case.
Seems it is more of just a polish issue - so I would prioritize it as a UX_P2.

Comment 5

[Casey Yee]

I run through this issue with LG and it seems that the problem is reproduced by:

1. reset data usage to 0
2. start browsing
3. open utility tray and note the data usage bar is overflowing the container.

We agree that it's a pretty minor issue, but it should be fixed.

Comment 6

[David Scravaglieri [:scravag]]

Please ask for approval once it is ready.

Comment 7

[Joan Leon [:joan]]

Attachment: Patch on GitHub

Add rule for fix it

Comment 8

[Arnau March [:arnau] ( not working in Firefox OS anymore :( )]

Comment on attachment 8377464 [details] [review]
Patch on GitHub

Awesome! Thanks Joan.

Comment 9

[Arnau March [:arnau] ( not working in Firefox OS anymore :( )]

Merged: e1f8ebe3cbe27923f0070bc13fc059b55749a9f1

Comment 10

[Joan Leon [:joan]]

Comment on attachment 8377464 [details] [review]
Patch on GitHub

NOTE: Please see https://wiki.mozilla.org/Release_Management/B2G_Landing to better understand the B2G approval process and landings.

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): missing property
[User impact] if declined: not matching UI specs
[Testing completed]: yes
[Risk to taking this patch] (and alternatives if risky): none
[String changes made]:

Comment 11

[Preeti Raghunath(:Preeti)]

HI,

This is not a blocker and hence will not be taking for 1.3

Comment 12

[[:fabrice] Fabrice Desré]

Comment on attachment 8377464 [details] [review]
Patch on GitHub

clearing approval since it's not a blocker.