Skip to content
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

🐞 Bug: input-radio akzeptiert keine stringifizierten Zahlenwerte #5986

Open
laske185 opened this issue Jan 31, 2024 · 9 comments
Open

🐞 Bug: input-radio akzeptiert keine stringifizierten Zahlenwerte #5986

laske185 opened this issue Jan 31, 2024 · 9 comments
Assignees
Labels
bug Something isn't working desybri The task is assigned to the DESYBRI design system. needs reproduction

Comments

@laske185
Copy link
Contributor

Fehlermeldung

Beschreibung des Fehlers

Sind die Werte eines input-radio stringifizierte Zahlenwerte, wird nicht der gesetzte Wert gesetzt.

Reproduktion

<KolInputRadio
  _label={'Radio group'}
  _value="310002"
  _options={[
    { label: 'Ja', value: '310000' },
    { label: 'Nein', value: '310001' },
    { label: 'Unbekannt', value: '310002' },
  ]}
/>

Wird der Wert alphanumerisch, funktioniert es:

<KolInputRadio
  _label={'Radio group 2'}
  _value="A310002"
  _options={[
    { label: 'Ja', value: 'A310000' },
    { label: 'Nein', value: 'A310001' },
    { label: 'Unbekannt', value: 'A310002' },
  ]}
/>

Link zu StackBlitz

Erwartetes Verhalten

Der gesetzte Wert von _value wird auch ausgewählt.

Screenshots

image

Zusätzlicher Kontext

Der Bug wurde intern gemeldet.

@laske185 laske185 added the bug Something isn't working label Jan 31, 2024
@laske185 laske185 moved this to 📄 Ready in KoliBri Board Jan 31, 2024
@deleonio
Copy link
Contributor

deleonio commented Feb 1, 2024

Das folgende Ticket kann man ggf. gut mit diesem zusammen bearbeiten: #5992

@laske185 laske185 added the good first issue Good for newcomers label Feb 1, 2024
@KawaiiMochiAhri
Copy link
Contributor

Ich würde mir das mal anschauen. Kann mich leider nicht auf das Ticket assignen, desshalb der Kommentar. :)

@deleonio deleonio added the community The task is implemented in a community-driven manner. label Feb 2, 2024
@sdvg sdvg moved this from 📄 Ready to 🏗 In progress in KoliBri Board Feb 5, 2024
@KawaiiMochiAhri KawaiiMochiAhri linked a pull request Feb 5, 2024 that will close this issue
@KawaiiMochiAhri
Copy link
Contributor

Also meine Lösung funktioniert soweit.
Frage ist nur, ob das so elegant ist. Und an einer Stelle könnte man bestimmt noch refaktorn, aber ich war mir nicht sicher, inwiefern die abfolge des Codes wichtig ist.

@deleonio deleonio added needs reproduction and removed good first issue Good for newcomers community The task is implemented in a community-driven manner. labels Feb 12, 2024
@deleonio
Copy link
Contributor

Ich konnte das Problem auf Ebene der Web Components nicht reproduziert. Alle Typen der bleiben stabil.

Wir sollten das Beispiel nun im React-Kontext ausprobieren.

  • v1:
  • v2:

@deleonio deleonio assigned deleonio and unassigned KawaiiMochiAhri Feb 12, 2024
@Chrisdo82
Copy link
Contributor

Chrisdo82 commented Apr 3, 2024

@deleonio in dem Beispiel oben von @laske185 entspricht der value bei beiden Beispielen nicht dem was mit _options mitgegeben wurde. Der erste Radiobutton hat einen value von -0, der nächste -1 usw.

Bildschirmfoto 2024-04-03 um 10 32 08

Hier #2152 wurde ja schonmal geschrieben, warum das so ist. Allerdings wird das onChange-Event nicht mehr vom InputRadio aufgerufen und onInput hat keinen Value. Somit funktioniert der "Workaround" auch nicht mehr.

In Version 2 von KoliBri werden wir auch statische Formulare voll unterstützen und dafür wird hier, wie Du auch erwartest, wieder genau der Wert stehen, den Du reingegeben hast. (Ist für static Forms nötig) Der Komfort für die App-Entwicklung wird wegfallen, kann aber mittels Formatter/Parsern vor Inputs leicht gehandled werden.

Lt. #2152 wolltet ihr ab v2.0 die Platzhalter-Values wieder entfernen. Gab es einen Grund warum das doch nicht passiert ist?

@deleonio deleonio added the desybri The task is assigned to the DESYBRI design system. label Apr 25, 2024
@deleonio
Copy link
Contributor

Hallo @Chrisdo82,

danke für den Hinweis aus dem Ticket #2152. Das Angleichen des Wertes im DOM und im JavaScript ist weiterhin der Plan.

Da der Wert bei Eingabefeldern mit KoliBri immer aus der onChange / onInput geholt werden soll, seit Anfang an, gab es keinen dringenden Grund die Änderung vorzuziehen.

Die Unterstützung statischer Formulare ist auch in Version 2 noch experimentell, wobei es schon zahlreich erfolgreich eingesetzt wird. Ausnahme sollte das Radio sein.

Hier haben wir auch eine Demo dazu: https://public-ui.github.io/v2/sample-react/#/scenarios/static-form


Was hindert Euch daran, einfach den Value aus den Events - onChange und onInput zu verwenden, wie das seit jeher funktioniert?

@Chrisdo82
Copy link
Contributor

Chrisdo82 commented May 2, 2024

Was hindert Euch daran, einfach den Value aus den Events - onChange und onInput zu verwenden, wie das seit jeher funktioniert?

@deleonio in der Version 2.0.12 wurde kein Value beim onInput mitgegeben, was die Komponente für uns unbrauchbar gemacht hat. Deshalb hatte ich hier auch nochmal dazu geschrieben. Mit der 2.0.14 geht es aber wieder.

@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in KoliBri Board May 21, 2024
@programmishka
Copy link

Bitte um Wiedereröffnung. Das beschriebene Beispiel funktioniert weiterhin nicht in der Komponenten-Version 2.2.1.

@deleonio deleonio reopened this Dec 6, 2024
@deleonio deleonio moved this from ✅ Done to 🗂️ Backlog in KoliBri Board Dec 6, 2024
@deleonio
Copy link
Contributor

deleonio commented Dec 6, 2024

Ist erneut geöffnet und prüfen dass kurz.

@deleonio deleonio removed the status in KoliBri Board Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working desybri The task is assigned to the DESYBRI design system. needs reproduction
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

5 participants