public-ui / kolibri

The accessible HTML-Standard
https://public-ui.github.io
European Union Public License 1.2
173 stars 34 forks source link

🐞 Bug: KolInputDate Fehlermeldung wird nicht zurückgesetzt #6977

Open Chrisdo82 opened 4 weeks ago

Chrisdo82 commented 4 weeks ago

Link to the code that reproduces this issue:

https://stackblitz.com/edit/vitejs-vite-qrazuc?file=src%2FApp.tsx

Can you categorise where the error occurs?

React

Which browser or operating system do you used to test KoliBri?

How to reproduce issue?

  1. Öffne https://stackblitz.com/edit/vitejs-vite-qrazuc?file=src%2FApp.tsx
    1. Klicke auf den "Toggle Error" Button
    2. Fehlermeldung wird angezeigt
    3. Klicke erneut auf den "Toggle Error" Button
    4. Fehlermeldung wird weiterhin angezeigt Bildschirmfoto 2024-10-28 um 10 11 26

Current vs. Expected:

Die Fehlermeldung sollte verschwinden, wenn _msg nicht definiert ist. Das tut sie derzeit jedoch nicht.

Environment informations:


 "Operating System": {
 "platform": "darwin",
 "arch": "arm64",
 "version": "23.6.0"
 },
 "Binaries": {
 "node": "v20.17.0",
 "npm": "10.8.2",
 "pnpm": "8.12.1",
 "yarn": "N/A"
 },
 "Relevant Packages": {
 "@public-ui/components": "2.1.8",
 "@public-ui/react": "2.1.8",
 "react": "^18.3.1",
 "react-dom": "^18.3.1",
 "typescript": "5.5.4"
 }
 } ```
Chrisdo82 commented 3 weeks ago

Betrifft vermutlich auch andere Inputfelder, ich konnte das gleiche Verhalten auch beim KolInputCheckbox nachstellen.

Chrisdo82 commented 1 week ago

@deleonio habt ihr euch das bereits anschauen können?

deleonio commented 1 week ago

@Chrisdo82 noch nicht, ich habe es jetzt schon einmal geplant.

deleonio commented 1 week ago

@laske185 dafür gibt es doch jetzt die reset()-Methode?

laske185 commented 1 week ago

Übergibt man null statt undefined wird die Meldung ausgeblendet, da undefined von Stencil ignoriert wird.

Bei der Property _error funktioniert es auch mit undefined: https://stackblitz.com/edit/vitejs-vite-7f2uql?file=src%2FApp.tsx

Todo:

deleonio commented 2 days ago

Should be works with the fix: https://stackblitz.com/edit/vitejs-vite-bserke?file=package.json,src%2FApp.tsx