Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix number inputs #4144

Merged
merged 3 commits into from
Mar 5, 2025
Merged

Conversation

robinlej
Copy link

@robinlej robinlej commented Mar 4, 2025

  • If you write "0 3 " in a number input, the last space creates an empty string which causes an error.
  • If there is no border, hasBorder returns an empty string which triggers an error on .match().

@robodoo
Copy link

robodoo commented Mar 4, 2025

This PR targets the un-managed branch odoo-dev/odoo:master-mysterious-egg, it needs to be retargeted before it can be merged.

@robinlej robinlej force-pushed the master-mysterious-egg-role-3 branch 2 times, most recently from 8c1e114 to a9c907d Compare March 4, 2025 15:06
robinlej added 3 commits March 4, 2025 17:41
If there is no border, `hasBorder` returns an empty string which
triggers an error on `.match()`.
If you write "0 3 " in a number input, the last space creates an empty
string which causes an error.
Only accept:
- numbers 0-9
- `.` and `,`
- `-` for negative values
- spaces to separate multiple values
@robinlej robinlej force-pushed the master-mysterious-egg-role-3 branch from 48ec8b5 to f1f88bf Compare March 4, 2025 16:42
@loco-odoo loco-odoo merged commit c3a825d into master-mysterious-egg Mar 5, 2025
@loco-odoo loco-odoo deleted the master-mysterious-egg-role-3 branch March 5, 2025 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants