Closed Bug 1023824 Opened 5 years ago Closed 5 years ago

[Homescreen] Label size should remain the same from 3 to 4 grid

Categories

(Firefox OS Graveyard :: Gaia::Homescreen, defect)

x86
Gonk (Firefox OS)
defect
Not set

Tracking

(Not tracked)

RESOLVED WORKSFORME

People

(Reporter: epang, Assigned: kgrandon)

Details

Attachments

(3 files)

Currently the labels on homescreen shrink in size when switching from the 3 to 4 icon grid.  The label size should remain the same in both.  

This also effects the search app since it follows the same label size as the homescreen.
This is done in CSS [1] but there was a problem with web components. I've installed the latest build and font sizes for apps seem the same thought. Could you double check it?

[1] https://github.com/mozilla-b2g/gaia/blob/master/shared/elements/gaia_grid/style.css#L59
Attached image homescreen grid.jpg
Hi Cristian, I've installed today's build and I'm still seeing it.  See the attach screen.  This is the build i flashed: https://pvtbuilds.mozilla.org/pvt/mozilla.org/b2gotoro/nightly/mozilla-central-flame/latest/

Is it the wrong one?  

Let me know, thanks!
Flags: needinfo?(crdlc)
I don't think I'd block the release on this. It's nice cleanup work, but not a critical blocker.
Blocks: vertical-home-next
No longer blocks: vertical-homescreen
Sorry no idea, I cannot open that URL because I am from Telefonica :) 
My build is gaia: master Gecko: 5cc2c0b.Gaia-981168c
A bug was filed regarding to font size and if you see the attachment in bug 1023817 you can see that the font size is the same now. Maybe Franceso can help you 

(In reply to Eric Pang [:epang] from comment #2)
> Created attachment 8438405 [details]
> homescreen grid.jpg
> 
> Hi Cristian, I've installed today's build and I'm still seeing it.  See the
> attach screen.  This is the build i flashed:
> https://pvtbuilds.mozilla.org/pvt/mozilla.org/b2gotoro/nightly/mozilla-
> central-flame/latest/
> 
> Is it the wrong one?  
> 
> Let me know, thanks!
upps I didn't add the ni?
Flags: needinfo?(crdlc) → needinfo?(francesco.lodolo)
I flashed that build, but it doesn't seem very updated.

Build ID: 20140610040208
Git commit: f42ebc93

On 4 columns I get the small font, on 3 columns the bigger font. Current builds get the same font, which causes bug 1023817.
Flags: needinfo?(francesco.lodolo)
Attached image Flame 4 columns
This is how the Flame with 4 columns currently looks if I flash the current Gaia on top of that image. 

Git version: 6b5be16c from 2014-06-11
Attached image 320x480 both grids
Maybe we have a problem with resolutions because in 320x480 works fine, Kevin, do you have a Flame to test it? Thanks
ni? myself to take a look.
Flags: needinfo?(kgrandon)
Sorry, not sure if my comment was clear: the build currently available for Flame (link to pvt) has that issue, but a current build of Gaia works as expected (font doesn't change).

Compare this (recent build)
https://bugzilla.mozilla.org/attachment.cgi?id=8438459

With the right one here
https://bug1023824.bugzilla.mozilla.org/attachment.cgi?id=8438405

I see the bigger font on both Keon and Flame, that's why I filed the other bug. For me this bugs is a WORKSFORME.
There was a bug before, but I just landed something yesterday to address this: https://github.com/mozilla-b2g/gaia/commit/4d2841771e6e2d62a4bdba93116ed7b27239a0b0

This should be resolved either with the latest master or the nightly build from today/tomorrow.
Status: NEW → RESOLVED
Closed: 5 years ago
Flags: needinfo?(kgrandon)
Resolution: --- → FIXED
Resolution: FIXED → WORKSFORME
(In reply to Kevin Grandon :kgrandon from comment #11)
> There was a bug before, but I just landed something yesterday to address
> this:
> https://github.com/mozilla-b2g/gaia/commit/
> 4d2841771e6e2d62a4bdba93116ed7b27239a0b0
> 
> This should be resolved either with the latest master or the nightly build
> from today/tomorrow.

Great, thanks guys!
No longer blocks: vertical-home-next
You need to log in before you can comment on or make changes to this bug.