Feature #5397
closed
Input type="number" erlaubt nur ganzzahlige Änderungen
Added by Elias Villiger about 6 years ago.
Updated about 5 years ago.
Description
Ich habe ein FormElement, welches als input type="number" gerendert wird (vermutl. aufgrund der Table-Definition des entspr. Feldes).
Dieser Input-Typ ist per Default auf step=1 gesetzt, was recht kuriose Folgen hat: Wenn ein Wert von 11.23 eingetragen ist, kann dieser zwar auf 12.23 oder 10.23 geändert werden, alle Werte dazwischen werden aber als ungültig gemeldet.
Lösungsvariante A: step="any" setzen - dies erlaubt die Angabe aller Zahlen (also z.B. auch 10.24); beim Klick auf die Spin-Buttons ist das tatsächliche Inkrement aber +/-1, entspricht also vermutlich den meisten use cases.
Lösungsvariante B: type="text" setzen
- Related to Bug #5390: Input Decimal: default sanitze class violated added
Hallo Elias
ja, 'number' ist ein neues Feature und seit #5390 habe ich 'type=number' wieder rausgenommen, bei allen ausser ganzzahlingen Typen, und auf den alten Wert 'type=text' gesetzt (=Deine Variante B).
Ein FE.parameter 'step' ist auf jeden Fall gut.
Mit #5390 sollte dieser Bug behoben sei (siehst Du das auch so?), aber 'step' macht trotzdem Sinn und sollte implementiert werden.
Danke
CU
Carsten
Ah, prima. Ja, tönt gut.
step Parameter macht aber nur Sinn, wenn auch input="number" verwendet wird.
input="number" sollte evtl auch bei ganzzahligen Typen nicht automatisch gerendert werden. Beispiel: ein typeAheadSql-Feld "xId" (Typ BIGINT) funktioniert nicht, wenn input="number" ist (falsches pattern und die spin buttons machen keinen Sinn).
Man könnte den input Type nur als Konfigurations-Option anbieten, wie bereits implementiert, und falls nicht angegeben input="text" verwenden.
- Related to Bug #5404: typeAheadSql: Pattern falsch erkannt, wenn Tabellen-Spalte Integer ist added
- Target version set to next8
- Tracker changed from Bug to Feature
- Status changed from New to In Progress
- Assignee changed from Carsten Rose to Elias Villiger
Todo: Add `step` attribute to FormElement parameters.
- Status changed from In Progress to Closed
Bereits implementiert (#6178)
- Target version changed from next8 to 19.2.1
Also available in: Atom
PDF
Updated by Carsten Rose 
Updated by Elias Villiger