Closed Bug 1030440 Opened 7 years ago Closed 7 years ago

[Vertical Homescreen] Optimize default icon configuration


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

Not set


(feature-b2g:-, b2g-v2.0 verified, b2g-v2.1 verified)

2.0 S5 (4july)
feature-b2g -
Tracking Status
b2g-v2.0 --- verified
b2g-v2.1 --- verified


(Reporter: jsavory, Assigned: kgrandon)



(Whiteboard: [systemsfe])


(6 files)

Attached image Icon_Layout.png
The default icon configuration could be improved by organizing the icon layout into certain groups and including more critical apps at the top of the screen. 

I've attached a proposal for a app icon layout that I think works a bit better than the current version.
feature-b2g: --- → -
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking+]
Whiteboard: [systemsfe]
Duplicate of this bug: 1029655
Good idea to do, but we'll ship even if this isn't finished.
Blocks: vertical-home-next
No longer blocks: vertical-homescreen
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking+] → [VH-FL-blocking-][VH-FC-blocking-]
Assignee: nobody → kgrandon
Target Milestone: --- → 2.0 S5 (4july)
Attached file Github pull request
This patch should fix the positioning of icons, though the collections are not respecting the default layout. I am investigating.
Comment on attachment 8448958 [details] [review]
Github pull request

Hey Cristian - 

Would you be able to review this first part of the placement work? There is more work to ensure collections are properly placed, and I'm going to open a new bug for that, as it seems fairly complex.
Attachment #8448958 - Flags: review?(crdlc)
Depends on: 1034364
Hmm, for 2.0 phones it seems people don't need bug 1029655 after flashing/updating, so I don't think it's needed. I think our default engineering build might just be wrong though.
No longer depends on: 1034364
Comment on attachment 8448958 [details] [review]
Github pull request

Attachment #8448958 - Flags: review?(crdlc) → review+
Blocks: vertical-homescreen
No longer blocks: vertical-home-next
Closed: 7 years ago
Resolution: --- → FIXED
Comment on attachment 8448958 [details] [review]
Github pull request

This is a nice low-risk patch that makes the default icon layout much nicer for users. We would like to uplift for the vertical homescreen.
Attachment #8448958 - Flags: approval-gaia-v2.0?(bbajaj)
Attachment #8448958 - Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
Depends on: 1058931
Depends on: 1058952
No longer depends on: 1058952
No longer depends on: 1058931
What is going on with the dependency fiddling?
Depends on: 1058987
No longer depends on: 1058987
Sorry everyone, typo on my part. I was blocking bugs to 1033040 and instead blocked it to this bug.

Apologize for the spam.
Hi Jsavory,
   Could you provide the detailed reproduce steps for me to verify this bug?

Thank you!
Flags: needinfo?(jsavory)
This is trivial to verify, just make sure that the icons match the layout according to the attachment here (Icon_Layout.png).
Flags: needinfo?(jsavory)
This issue has been successfully verified on Flame 2.1&2.0.
See attachment: verify_2.1_1.png,verify_2.1_2.png,verify_2.1_3.png and verify_2.0_4.png.
Reproduce rate: 0/3

Flame 2.0 versions:
Gaia-Rev        8d1e868864c8a8f1e037685f0656d1da70d08c06
Build-ID        20141130000204
Version         32.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141130.032432
FW-Date         Sun Nov 30 03:24:44 EST 2014
Bootloader      L1TC00011880

Flame 2.1 versions:
Gaia-Rev        ccb49abe412c978a4045f0c75abff534372716c4
Build-ID        20141130001203
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141130.034738
FW-Date         Sun Nov 30 03:47:49 EST 2014
Bootloader      L1TC00011880
You need to log in before you can comment on or make changes to this bug.