odoo icon indicating copy to clipboard operation
odoo copied to clipboard

[REF] point_of_sale: simplify numpad tour util

Open vlst-odoo opened this issue 1 year ago • 1 comments

When simply using :contains for targeting a specific button on the numpad we might run into the issue where doing :contains('1') targets the +10 button instead of the 1 button.

To solve this we currently rely on overly complicated js code.

In this commit we simplify the code by taking advantage of regex.

Task: 4214042


I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

vlst-odoo avatar Sep 26 '24 14:09 vlst-odoo

Pull request status dashboard

robodoo avatar Sep 26 '24 14:09 robodoo

Nice. I didn't know that contains works with regex.

caburj avatar Oct 10 '24 16:10 caburj

robodoo r+

caburj avatar Oct 10 '24 16:10 caburj

Nice. I didn't know that contains works with regex.

indeed, only the hoot version does

vlst-odoo avatar Oct 11 '24 08:10 vlst-odoo