[IMP] web: show % symbol outside the input in percentage widget

Before this commit the % symbol was part of the input. We could write a
value with the % symbol to divide the value by 100 (50% = 0.5, 50 = 50)

This commit removes the % symbol from the input and adds it in a span
after the input so we do not need to write the % symbol anymore.

The value from the input will be saved as $value / 100 in database.

task-2065078

closes odoo/odoo#43995

Signed-off-by: Aaron Bohy (aab) <aab@odoo.com>
This commit is contained in:
Michael Mattiello (mcm)
2020-02-06 11:55:13 +00:00
parent 9d85bce7db
commit c5e34d77f5
5 changed files with 91 additions and 22 deletions
@@ -904,6 +904,46 @@ QUnit.module('basic_fields', {
form.destroy();
});
QUnit.module('Percentage');
QUnit.test('percentage widget in form view', async function (assert) {
assert.expect(6);
const form = await createView({
View: FormView,
model: 'partner',
data: this.data,
arch: ` <form string="Partners">
<field name="qux" widget="percentage"/>
</form>`,
mockRPC: function (route, args) {
if (args.method === 'write') {
assert.strictEqual(args.args[1].qux, 0.24, 'the correct float value should be saved');
}
return this._super(...arguments);
},
res_id: 1,
});
assert.strictEqual(form.$('.o_field_widget').first().text(), '44.4%',
'The value should be displayed properly.');
await testUtils.form.clickEdit(form);
assert.strictEqual(form.$('.o_field_widget[name=qux] input').val(), '44.4',
'The input should be rendered without the percentage symbol.');
assert.strictEqual(form.$('.o_field_widget[name=qux] span').text(), '%',
'The input should be followed by a span containing the percentage symbol.');
await testUtils.fields.editInput(form.$('.o_field_float_percentage input'), '24');
assert.strictEqual(form.$('.o_field_widget[name=qux] input').val(), '24',
'The value should not be formated yet.');
await testUtils.form.clickSave(form);
assert.strictEqual(form.$('.o_field_widget').text(), '24%',
'The new value should be formatted properly.');
form.destroy();
});
QUnit.module('FieldEmail');