Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(127)

Unified Diff: chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js

Issue 2260653002: Pin keyboard works with virtual keyboard. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@lkgr
Patch Set: Nit. Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js
diff --git a/chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js b/chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js
index b35fcaa44403b899e4b012b55563ceba8baa7b2a..d6c28711b4d28f8981da2fa4aa487e02dafc6e40 100644
--- a/chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js
+++ b/chrome/browser/resources/chromeos/quick_unlock/pin_keyboard.js
@@ -40,9 +40,9 @@ Polymer({
},
/**
- * Whether or not the keyboard's submit button should be shown.
+ * Whether or not the keyboard's input should be shown.
*/
- enableSubmitButton: {
+ hideInput: {
type: Boolean,
value: false
},
@@ -77,13 +77,6 @@ Polymer({
onNumberTap_: function(event, detail) {
var numberValue = event.target.getAttribute('value');
this.value += numberValue;
-
- // If a number button is clicked, we do not want to switch focus to the
- // button, therefore we transfer focus back to the input, but if a number
- // button is tabbed into, it should keep focus, so users can use tab and
- // spacebar/return to enter their PIN.
- if (!event.target.receivedFocusFromKeyboard)
- this.focus();
},
/** Fires a submit event with the current PIN value. */
@@ -102,7 +95,8 @@ Polymer({
this.fire('pin-change', { pin: value });
},
- /** Called when the user wants to erase the last character of the entered
+ /**
+ * Called when the user wants to erase the last character of the entered
* PIN value.
*/
onPinClear_: function() {
@@ -127,6 +121,41 @@ Polymer({
},
/**
+ * Keypress does not handle backspace but handles the char codes nicely, so we
+ * have a seperate event to process the backspaces.
+ * @param {Event} event Keypress Event object.
+ * @private
+ */
+ onKeyDown_: function(event) {
+ // Backspace pressed.
+ if (event.keyCode == 8) {
+ this.onPinClear_();
+ event.preventDefault();
+ return;
+ }
+ },
+
+ /**
+ * Called when a key press event is fired while the number button is focused.
+ * Ideally we would want to pass focus back to the input element, but the we
+ * cannot or the virtual keyboard will keep poping up.
+ * @param {Event} event Keypress Event object.
+ * @private
+ */
+ onKeyPress_: function(event) {
+ var code = event.keyCode;
+
+ // Enter pressed.
+ if (code == 13) {
+ this.firePinSubmitEvent_();
+ event.preventDefault();
+ return;
+ }
+
+ this.value += String.fromCharCode(code);
+ },
+
+ /**
* Disables the submit and backspace button if nothing is entered.
* @param {string} value
* @private

Powered by Google App Engine
This is Rietveld 408576698