Fix TextEdit v_scroll_speed invalid values breaks wheel scrolling

[jmb462]

Fixes #58009

Issue description

Some values of text_editor/behavior/navigation/v_scroll_speed Editor settings breaks mouse wheel scrolling of TextEdit.

Identified causes

• Missing hint_range in property declaration to prevent null or negative values
• If the value is under 8, the increment in vscroll position is under its range step, so the value keeps unchanged and wheel scroll is blocked.

Fix proposal

• Add hint_range "1, 10000,1" in property declaration
  10000 leads to immediately scroll to destination. As comparison, clicking on minimap leads to a 3000 scrolling speed and is already nearly instantaneous.

• Ensure that scrolling speed is equal or greater than the scrollbar step to prevent blocking mouse wheel scroll.

After (I roll the mouse wheel a LOT to see the scrolling speed)

[bruvzg]

In addition to the property hint, it's probably worth adding the same checks to the TextEdit::set_v_scroll_speed and reject invalid values. Property hint will help when it's set from UI, but won't if it's set in the config file or set_v_scroll_speed is called directly.

[jmb462]

For too low values, should we set it to the minimum accepted value or should we return without modifying the value and throw an error/warning ? (I've pushed the first way but a doubt has raised now)

Higher values are not really a problem. It will be "instant" scroll.

[Paulb23]

Would suggest using ERR_FAIL_COND rather then MAX otherwise looks good.

[jmb462]

Would suggest using ERR_FAIL_COND rather then MAX otherwise looks good.

You're right. Done.

[akien-mga]

Thanks!