Techsystech
/
web
mirror of https://github.com/OCA/web.git
3
0
Fork 1
Code Issues Packages Projects Releases Wiki Activity

[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/1402/head
Jairo Llopis 2019-01-18 07:33:45 +00:00 committed by emagdalena
parent 166cd74f80
commit d096c0fbba
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -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"),