1057997 - [Sora][Monitor][Home screen]Home screen icons overlap after standby for a long time

Status: RESOLVED DUPLICATE of bug 995886

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

Description

[sync-1]

DEFECT DESCRIPTION:
 FFOS 1.3.
 
  Home screen icons overlap after standby for a long time
 
  REPRODUCING PROCEDURES:
  1.device standby one night
  2.unlock the device find icons overlap->ko
 
  EXPECTED BEHAVIOUR:
  KO:icons don't overlap
 
  ASSOCIATE SPECIFICATION:
 
  TEST PLAN REFERENCE:
 
  TOOLS AND PLATFORMS USED:
 
  USER IMPACT:
  High
 
  REPRODUCING RATE:
  2/16
 
  For FT PR, Please list reference mobile's behavior:

Comment 1

[sync-1]

Attachment: jrdlog.zip

Comment 2

[sync-1]

Attachment: VID_20140806_092352.3gp

Comment 3

[sync-1]

Attachment: IMG_20140806_123709.jpg

Comment 4

[sync-1]

Attachment: Pic

Comment 5

[sync-1]

Attachment: video

Comment 6

[sync-1]

Attachment: log

Comment 7

[sync-1]

Attachment: Pic

Comment 8

[sync-1]

Dears,
Sometimes when homescreen start, 
var windowWidth = window.innerWidth; (grid.js) windowWidth is 0, so cause this problem.

Comment 9

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Looks like an interesting bug...Dear Evelyn, mind finding someone to take a look at this issue?

Thanks for your help

Comment 10

[Evelyn Hung]

It's a dup of bug 995886 which we already fixed on 1.3t and 1.4.

Comment 11

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

(In reply to Evelyn Hung [:evelyn] from comment #10)
> It's a dup of bug 995886 which we already fixed on 1.3t and 1.4.
> 
> *** This bug has been marked as a duplicate of bug 995886 ***

Thanks Evelyn, i will have partners to cherry-pick the patch

Comment 12

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

Hi Evelyn -

Got a quick question: Seems like we change the status bar height since v2.0. So should we modify the following patch?

+ // 20px statusbar (2rem * 10px see system/style/statusbar/statusbar.css)
+ height = screen.height - 20;

Thanks for your help

Vance

Comment 13

[Evelyn Hung]

Hi Vance,
We've resolved the issue since 1.3t and the following release includes 2.0, so I'm not sure why our partner need this. If they are trying to fix 1.3, then please pick the patch for 1.3t or 1.4. Thanks.

Comment 14

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

(In reply to Evelyn Hung [:evelyn] from comment #13)
> Hi Vance,
> We've resolved the issue since 1.3t and the following release includes 2.0,
> so I'm not sure why our partner need this. If they are trying to fix 1.3,
> then please pick the patch for 1.3t or 1.4. Thanks.

Thanks Evelyn, in fact, TCL found this issue on their 1.3 devices, that's why they submitted this bug, and then they saw your Comment#10 and they take a look at the patch. the patch for 1.3T and 1.4 is OK, but they have some questions about the patch of master, I know the patches is already in 2.0 and partner don't need to do anything, but they just wonder why the height can still be 20 since the status bar already change the definition.

Sorry about the confusion, if there is any, and also ni Chen Guoqiang from TCL if need further explanation

Comment 15

[sync-1]

Dear Evelyn,

I'm confuse for 2.0 patch, because the status bar height should be 24, not 20

Comment 16

[Evelyn Hung]

(In reply to sync-1 from comment #15)
> Dear Evelyn,
> 
> I'm confuse for 2.0 patch, because the status bar height should be 24, not 20

Oh, then I suggest you comment on that bug to ask patch author. It's better than my guess. :)