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: Regression bei onChange-Listener der KolInputRadio-Komponente #6416

Closed
oas opened this issue May 13, 2024 · 4 comments
Closed

🐞 Bug: Regression bei onChange-Listener der KolInputRadio-Komponente #6416

oas opened this issue May 13, 2024 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@oas
Copy link

oas commented May 13, 2024

Note

  • Ich achte beim Erstellen des Issues darauf, dass ich keine fachlichen und unternehmensinternen Details preisgebe.

Fehlermeldung

Beschreibung des Fehlers

In unserer Anwendung werden KolInputRadio-Komponenten basierend auf vorherigen Eingaben ein- oder ausgeblendet. Es scheint so, als ob der onChange-Listener der KolInputRadio-Komponente in einigen Fällen nicht mehr korrekt ausgelöst wird. Es handelt sich dabei um eine Regression, in älteren Versionen (bspw. 2.0.9) funktionierte es noch wie erwartet.

Reproduktion

Schritte zum Reproduzieren des Verhaltens:

  • "toggle"-Button auswählen, anschließend erscheint die KolInputRadio-Komponente
  • wie erwartet wird der onChange-Listener bei Auswahl eines Elements korrekt ausgelöst
  • "toggle"-Button zwei Mal auswählen, sodass die KolInputRadio-Komponente verschwindet und anschließend wieder auftaucht
  • der onChange-Listener wird bei Auswahl eines Elements nicht mehr ausgelöst

Erwartetes Verhalten

In der Konsole erscheint die Ausgabe "Selected value(s)" auch nach mehrmaliger Ein- / Ausblendung ohne Probleme.

Screenshots

Bildschirmaufnahme.2024-05-13.um.14.15.37.webm

Desktop:

  • Betriebssystem: macOS
  • Browser Chrome
  • Version 124.0.6367.158
@oas oas added the bug Something isn't working label May 13, 2024
@sdvg
Copy link
Member

sdvg commented May 21, 2024

Der Fehler ist in develop anscheinend schon behoben und sollte mit der nächsten KoliBri-Version dann nicht mehr auftauchen.
Ich werde das nach dem nächsten Release noch einmal prüfen und das Ticket dann ggf. schließen.

@deleonio
Copy link
Contributor

Der Fehler ist in develop anscheinend schon behoben und sollte mit der nächsten KoliBri-Version dann nicht mehr auftauchen. Ich werde das nach dem nächsten Release noch einmal prüfen und das Ticket dann ggf. schließen.

Prüfe bitte kurz folgende Änderung hinsichtlich dieses Tickets: 4ee2005

@sdvg
Copy link
Member

sdvg commented May 21, 2024

Ich habe es noch mal nachgestellt und kann bestätigen, dass der Bug auftritt, sobald man das beforePatch wieder herausnimmt 👍

@sdvg
Copy link
Member

sdvg commented May 29, 2024

Das Problem ist in der neuesten KoliBri-Version wie erwartet behoben.

Hier ein aktualisierter StackBlitz, indem ich bloß die neuesten KoliBri-Pakete installiert habe: https://stackblitz.com/edit/kolibri-bug-05-13-tvmksd

@sdvg sdvg closed this as completed May 29, 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
Projects
Status: ✅ Done
Development

No branches or pull requests

3 participants