-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Unable to enter decimal values into certain properties, including Umb.PropertyEditorUi.Decimal
.
#18150
Comments
Hi there @markadrake! Firstly, a big thank you for raising this issue. Every piece of feedback we receive helps us to make Umbraco better. We really appreciate your patience while we wait for our team to have a look at this but we wanted to let you know that we see this and share with you the plan for what comes next.
We wish we could work with everyone directly and assess your issue immediately but we're in the fortunate position of having lots of contributions to work with and only a few humans who are able to do it. We are making progress though and in the meantime, we will keep you in the loop and let you know when we have any questions. Thanks, from your friendly Umbraco GitHub bot 🤖 🙂 |
Note that the issue seems to be with this component: umb-property-editor-ui-number. |
@markadrake Thank you for reporting this, I appreciate the frustration it can cause. There are a few issues to resolve with this, some are in the CMS, others are in the UUI component library. There is a couple of things in the property-editor itself: changing from the The bug about the decimal indicator, I've identified the cause in the |
Hi @markadrake, can you confirm what the Step size value is on the data-type configuration for the Decimal editor? If it's From reading up about how the |
Fixed with umbraco/Umbraco.UI#1004 and due in 15.3 (though with the proviso that one further issue exists in the underlying UI library, which is raised as umbraco/Umbraco.UI#1004). |
Which Umbraco version are you using? (Please write the exact version, example: 10.1.0)
15.1.2
Bug summary
Typing decimal values into the
Umb.PropertyEditorUi.Decimal
property editor is challenging.It also results in a quasi-error state.
Configuration and result below:
Specifics
0
, and this value fills the box on first load..123
to create the value0.123
, the text box does not accept the.
character and creates1230
instead.backspace
(and thenbackspace
again because a0
is inserted) I can now type the decimal portion of my number.123
.I can save and publish this now. The field will be highlighted in red, indicating a validation error, but the value is saved to the database and retained after a reload.
Steps to reproduce
The steps to reproduce this issue have already been included in the summary and specifics section.
Expected result / actual result
0
is frustrating, even when backspacing. I question why we chose to do anything on the "input" event. Should we apply a validation pattern instead?#parseInt
method, which is inappropriately named as it only verifies that the content is a number.The text was updated successfully, but these errors were encountered: