Inspector no longer interprets Comma as decimal point in editor

[kondelik]

Godot version:
3.1-alpha (2 and 3, didnt test alpha1)

OS/device including version:
Windows 10, non-english keyboard

Issue description:

Regression of issue #6028 in godot 3.1 alpha. Original issue fixed by PR #13240

In GODOT 3.0 editor you can use "," as decimal separator:

(comma is simply replaced by dot)

In GODOT 3.1-alpha (2 and 3, didnt test 1) everything after comma is trimmed:

Steps to reproduce:

Same as in issue #6028

1. In editor, write "1.1" to any float box (like "rotation degrees" on Node2D) - rotation is set to "1.1"
2. In editor, write "1,1" to any float box (like "rotation degrees" on Node2D) - rotation is set to "1"

[akien-mga]

See discussion on #24532.

Simply replacing "," by "." is problematic as the EditorSpinSlider uses the Expression node to parse commands and can support functions with multiple arguments, such as pow(2, 3).

A better solution would be to intercept the key event from this numpad comma separator and replace it by a KEY_KP_PERIOD event (only in EditorSpinSlider). This implies actually exposing this separator event to Godot in platform keyboard bindings.

[Calinou]

@akien-mga Is using pow() and other functions in the inspector common enough? If not, I think it'd be better to just replace , with . at this point.

Most expression usage in the editor seems to revolve around basic math operators (+-*/%). If we really need pow(), I guess we could add an ** exponentation operator to Expression.

[akien-mga]

It's important in Expression yes, which can parse full expressions.

That's why I said this should be handled ad-hoc in the inspector only (EditorSpinSlider).

[Calinou]

Closing in favor of godotengine/godot-proposals#1576, as feature proposals are now tracked in the Godot proposals repository.