Browse Source

[FIX] web_decimal_numpad_dot: Restore keyboard navigation

When https://github.com/OCA/web/pull/1154 switched to using `keydown` as the event to trigger the numpad dot replacement, it was actually [disabling upstream's `_onKeyDown()`][1], which in turn disabled the keyboard navigation.

To fix this situation, I replaced the method that is being used to directly override `_onKeyDown`.

[1]: 19558129f0/addons/web/static/src/js/fields/abstract_field.js (L41)
pull/1174/head
Jairo Llopis 6 years ago
parent
commit
67f76e8fab
No known key found for this signature in database GPG Key ID: 59564BF1E22F314F
  1. 11
      web_decimal_numpad_dot/static/src/js/numpad_dot.js

11
web_decimal_numpad_dot/static/src/js/numpad_dot.js

@ -7,13 +7,6 @@ odoo.define("web_decimal_numpad_dot.FieldFloat", function (require) {
var translation = require("web.translation"); var translation = require("web.translation");
var NumpadDotReplaceMixin = { var NumpadDotReplaceMixin = {
init: function () {
this.events = $.extend({}, this.events, {
"keydown": "numpad_dot_replace",
});
return this._super.apply(this, arguments);
},
l10n_decimal_point: function () { l10n_decimal_point: function () {
return this.formatType === "float_time" return this.formatType === "float_time"
? ":" : translation._t.database.parameters.decimal_point; ? ":" : translation._t.database.parameters.decimal_point;
@ -24,10 +17,10 @@ odoo.define("web_decimal_numpad_dot.FieldFloat", function (require) {
cur_val.substring(to); cur_val.substring(to);
}, },
numpad_dot_replace: function (event) {
_onKeydown: function (event) {
// Only act on numpad dot key // Only act on numpad dot key
if (event.keyCode !== 110) { if (event.keyCode !== 110) {
return;
return this._super.apply(this, arguments);
} }
event.preventDefault(); event.preventDefault();
var from = this.$input.prop("selectionStart"), var from = this.$input.prop("selectionStart"),

Loading…
Cancel
Save