Closed Bug 1086215 Opened 10 years ago Closed 10 years ago

[LockScreen] Resume the animation delay of Bug 1069879 after Bug 1079706 landed

Categories

(Firefox OS Graveyard :: Gaia::System::Lockscreen, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gweng, Assigned: gweng)

References

Details

Attachments

(1 file)

46 bytes, text/x-github-pull-request
timdream
: review+
Details | Review
Since Bug 1069879 follows the old way to control the panel animation, which would be eliminated with Bug 1079706, we need to re-implement the method of animation delaying according to the new mechanism.

The reason I don't want to resolve it in Bug 1079706 is Bug 1069879 is a 2.1 blocker with 2 patches for master and 2.1, but the change only happen on master. So I fire this bug to resolve it.
Assignee: nobody → gweng
Depends on: 1079706
Attached file Patch
1. Move the unlocking logic of passcode-enabled unlocking from lockscreen.js to the new Unlock state.

2. Map input-window-close/open events to state transferring, and replace the previous way (state managed) with the events version.

3. Fix the relevant tests.
Attachment #8512501 - Flags: review?(timdream)
The 1. is the major issue we must fix because before this patch, the unlocking logic and its animation is out of states' control, so the animation performs concurrently, and caused the bug.
Attachment #8512501 - Flags: review?(timdream) → review+
master: https://github.com/mozilla-b2g/gaia/commit/678fd2281b678e665c1a77eb4d6325a626768679
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: