Browse Source

fix tab navigation issue in web_decimal_numpad_dot

pull/1097/head
Jordi Ballester Alomar 6 years ago
committed by Pedro M. Baeza
parent
commit
56f7bad1c3
  1. 2
      web_decimal_numpad_dot/__manifest__.py
  2. 14
      web_decimal_numpad_dot/static/src/js/numpad_dot.js

2
web_decimal_numpad_dot/__manifest__.py

@ -8,7 +8,7 @@
{ {
"name": "Web - Numpad Dot as decimal separator", "name": "Web - Numpad Dot as decimal separator",
"version": "11.0.1.0.1",
"version": "11.0.1.0.2",
"license": "AGPL-3", "license": "AGPL-3",
"summary": "Allows using numpad dot to enter period decimal separator", "summary": "Allows using numpad dot to enter period decimal separator",
"depends": [ "depends": [

14
web_decimal_numpad_dot/static/src/js/numpad_dot.js

@ -9,7 +9,7 @@ odoo.define("web_decimal_numpad_dot.FieldFloat", function (require) {
var NumpadDotReplaceMixin = { var NumpadDotReplaceMixin = {
init: function () { init: function () {
this.events = $.extend({}, this.events, { this.events = $.extend({}, this.events, {
"keydown": "numpad_dot_replace",
"keyup": "numpad_dot_replace",
}); });
return this._super.apply(this, arguments); return this._super.apply(this, arguments);
}, },
@ -20,7 +20,11 @@ odoo.define("web_decimal_numpad_dot.FieldFloat", function (require) {
}, },
numpad_dot_replace: function (event) { numpad_dot_replace: function (event) {
String.prototype.replaceAt=function(index, replacement) {
return this.substr(0, index) + replacement + this.substr(index + replacement.length);
}
// Only act on numpad dot key // Only act on numpad dot key
event.stopPropagation()
if (event.keyCode != 110) { if (event.keyCode != 110) {
return; return;
} }
@ -29,12 +33,8 @@ odoo.define("web_decimal_numpad_dot.FieldFloat", function (require) {
to = this.$input.prop("selectionEnd"), to = this.$input.prop("selectionEnd"),
cur_val = this.$input.val(), cur_val = this.$input.val(),
point = this.l10n_decimal_point(); point = this.l10n_decimal_point();
// Replace selected text by proper character
this.$input.val(
cur_val.substring(0, from) +
point +
cur_val.substring(to)
);
var new_val = cur_val.replaceAt(from-1, point)
this.$input.val(new_val);
// Put user caret in place // Put user caret in place
to = from + point.length to = from + point.length
this.$input.prop("selectionStart", to).prop("selectionEnd", to); this.$input.prop("selectionStart", to).prop("selectionEnd", to);

Loading…
Cancel
Save