Closed Bug 913552 Opened 8 years ago Closed 8 years ago
[Cost Control] Grid numbers are too close to the left edge of screen
Hi, The numbers on the left of the grid are too close to the left edge of the screen in SD and HD versions. Can this be adjusted so there's more space on the left side? Thanks
Summary: [SMS] Header graphic and tab not aligned → [Cost Control] Grid numbers are too close to the left edge of screen
Hí José, could you give some feedback about this bug? Regards
Hi, I was checking the IxD specs and this issue seems to be a wrong implementation of the MB indicator. Please check the page 23 of the latest spec: https://mozilla.box.com/s/k3h8p2rm6ldgd7ogfurj "MB and GB indicators only positioned at unit change" in this case the "0 B" can be replaced with "0 MB", is the same information but the spacing issue will be solved.
In this patch I implement the following changes asked by José Vittone: - Graph legend changes: * The unit showed on the first label (0) must be the same that the first scale step. Eg. Actual scale (0B - 620KB - 1.24MB - 1.86 - 2.48 - 3.1) --> New Scale (0KB - 620 - 1.24MB - 1.86 - 2.48 - 3.1). * The left legend must have a normal weight (400) the actual is 600. * The numbers on left side only have one decimal position when its value is lower than 10, otherwise the number must be represented without decimals. Eg. Actual scale (0B - 620KB - 1.24MB - 1.86 - 2.48 - 3.1) --> New Scale (0KB - 620 - 1.2MB - 1.9 - 2.5 - 3.1) * Now, not always the last vertical line of the right edge was drawn, this line must be drawn always. * On the bottom of the chart, the months mus be capitalized. (its first letter as a upper-case letter and the remaining letters in lower case). * On the bottom of the chart, the today fontsize must be the same of the month legend. - On the bottom of the graphic (toggle data): * The letters are not showed completed, the tails on descending letters - "q, p, y" etc - disappear. (On both labels - "Mobile Usage" and "Wi-Fi Usage") Salva, do you mind reviewing the patch? Regards
Attachment #8385276 - Flags: review?(salva)
I've asked marina to implement some small visual deviations on cost control page. Please, take a look at the attached image. A) there is a missing line on the right side of the graph. B) Today's label has a different font size and it should not. B) & C) Following the visual specs, dates shouldn't be all caps. Instead they should be like “Mar 1”. D) if you take a look that lower parts of the letters, they are been cut. For instance, take a look at letter "g".
Attachment #8385276 - Flags: review?(salva) → review?(jmcf)
Comment on attachment 8385276 [details] [review] patch v1.0 We need another round, please ask me once it is ready
Comment on attachment 8385276 [details] [review] patch v1.0 Updated the PR with your comments.
Comment on attachment 8385276 [details] [review] patch v1.0 thanks, merge once Travis is green
Attachment #8385276 - Flags: review?(jmcf) → review+
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.