Closed
Bug 1053680
Opened 10 years ago
Closed 7 years ago
[LockScreen][Keyboard] Investigate if LockScreen can use the existing keyboard app instead of the keypad element
Categories
(Firefox OS Graveyard :: Gaia::System::Lockscreen, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: gweng, Unassigned)
References
Details
To avoid update two codes and styles, we should try to use existing keyboard in LockScreen.
The possible code path is:
1. LockScreenWindowManager know now should show the keyboard on LockScreen
2. Fire system-resize to resize the active LockScreenWindow
3. Stop the event propagation
4. The LockScreenWindow get resized to show the keyboard frame
The OOM issue should be considered, to avoid leaking information while the passcode shows.
Comment 1•10 years ago
|
||
Some follow-ups:
1. Experimentation: I added a <input type="text"> to lockscreen without changing JS codes and allowed keyboard to have higher z-index than lockscreen. On input's receiving/losing focus, The keyboard and input management seemed to behave quite well and as normally as we'd expect [1].
2. According to bug 1060193 comment 1, in Settings app's Lockscreen passcode panel, what we see as apparently receiving inputs are actually <span>s, and there is a hidden input there to take keyboard inputs. We probably can copy-cat this model in Lockscreen.
3. We'd also need to consider how we're going to overlay "Emergency Call" key and "Cancel" key and have them behave expectedly. Overlaying "Cancel" is trickier as it can visually collide with backspace key on the keyboard.
[1] this was two weeks ago; ongoing Lockscreen features/refactoring works may have changed things.
By the way, do we really want to resize the lockscreen window? (if we just want correct z positions maybe just messing around with CSS would be easier).
Comment 2•7 years ago
|
||
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•