Problem summary
Numeric fields where thousands separator commas are input incorrectly won't be automatically corrected and won't update the setting.
NOTE: In Mari, the full stop punctuation mark is used as the decimal separator, while commas are used as the thousands separator. For example, the number one million is presented as 1,000,000.000
Customer reported version
Mari 7.0v2
Steps to reproduce
This affects any field in Mari that expects a number, the following is just one example:
- Launch Mari and open any Project, such as the Example Project.
- Change to the Perspective tab.
- In the Canvas toolbar, set the Far setting to, for example, 1,000.000
(which is the default value)
- Move the camera just far enough from the Object so that it gets clipped and disappears.
- Add a 0 to the Far setting with the intention of making the Object visible without moving the camera. However, add it so that the resulting input is 1,0000.000
- Press [Enter] and/or click out of the setting.
Expected behaviour
Two things should happen:
- The incorrect value in the numeric input should be automatically edited to use commas correctly as 10,000.000
Alternatively, Mari should do what it used to before Mari 6.0v1, and just revert to the last correct input, which in this case would be 1,000.000
- The Far setting should correctly update, and the Object would become visible in the Canvas.
Actual behaviour
Nothing happens. When pressing [Enter], the caret remains active in the input field, the number doesn't change, and the setting doesn't update.
Workaround
The whole existing number can be removed and the new number input without commas as 10000. In this case, Mari will correctly add the commas (and decimal point) automatically upon pressing [Enter], and the setting will be updated.
Reproduced by support
This bug has been reproduced in:
Mari 7.0v2 - Windows 10, Rocky 9
Mari 6.0v1 - Windows 10, CentOS 7 - Regression
Unable to reproduce bug in:
Mari 5.0v4
Earliest version tested
Mari 5.0v4 - This issue doesn't appear in this version and has regressed
NOTE: The use of commas as thousands separators was added in Mari 5.0v3.