878732 - Homescreen gets stuck between everything.me screen and time display

Status: RESOLVED FIXED

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

Description

[Edwin Flores [inactive from 2016-12-01] [:eflores] [:edwin]]

On nightly branch

STR:
1. Restart b2g
2. At homescreen time display, swipe right to start e.me loading
3. While e.me screen is loading, swipe left to partially reveal time display and hold in that position until e.me is finished loading
4. Release

Expected:
Homescreen should spring back to either time display or e.me

Actual:
Homescreen gets stuck between the two screens

Comment 1

[Cristian Rodriguez (:crdlc)]

Attachment: Patch v1

Comment 2

[Evyatar 'Tron' Amitay (everything.me)]

cristian- I understand the patch and it's a cool and easy solution- but it doesn't seem to work on my device (Unagi). I can still when loading...

Comment 3

[Cristian Rodriguez (:crdlc)]

umm I cannot reproduce it, I gonna take a look tomorrow better, thanks

Comment 4

[Evyatar 'Tron' Amitay (everything.me)]

Cristian, any insights?
Do you prefer us at e.me just take it?

Comment 5

[Cristian Rodriguez (:crdlc)]

Hi, I gonna try to reproduce it during this morning if I cannot, I'll pass you the bug :) Thanks

Comment 6

[Cristian Rodriguez (:crdlc)]

It works fine for me with this patch, could you provide me with new STR? Thanks a lot

Comment 7

[Evyatar 'Tron' Amitay (everything.me)]

ok you're right- after the patch, when holding the swipe between e.me and landing, and then releasing- it fixes itself correctly.

Comment 8

[Cristian Rodriguez (:crdlc)]

https://github.com/mozilla-b2g/gaia/commit/a77a4185859c58b78eff85d0ed6a738184685eaa

Comment 9

[Cristian Rodriguez (:crdlc)]

Comment on attachment 757401 [details]
Patch v1

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 #): 
User impact if declined: low because IMHO the STR are weird for common final users and furthermore they can recover the state touching the screen but the reporter demonstrated that is possible and how the risk is null I would like to add to v1-train branch
Testing completed: yes
Risk to taking this patch (and alternatives if risky): null, only one CSS rule was added so I considered that it could be on v1-train
String or UUID changes made by this patch:

Comment 10

[Alex Keybl [:akeybl]]

Comment on attachment 757401 [details]
Patch v1

Approvals are now closed. Please nominate for blocking-b2g:leo? for critical issues.

Comment 11

[Cristian Rodriguez (:crdlc)]

It is not critical but the risk is null and nice to have

Comment 12

[Wayne Chang [:wchang]]

Triage - low impact, easy recovery, nonblocking.