Sicherstellen, dass alle Event-Listener korrekt funktionieren
Akzeptanzkriterien
Keine doppelten Funktionsdefinitionen mehr
Keine ReferenceError in der Console
Lightbox funktioniert weiterhin (Klick + Keyboard)
Floor-Accordion funktioniert weiterhin
## Problem
Aus der Schwachstellenanalyse ergeben sich zwei zusammenhängende JS-Probleme in `js/haus-schleusingen.js`:
### 1. Doppelte Funktionsdefinitionen
`openLightbox()` und `closeLightbox()` werden **zweimal** definiert – einmal mit jQuery, einmal vanilla. Die zweite Definition überschreibt die erste.
### 2. Toter Code im Keyboard-Handler
Außerhalb jedes Event-Listeners steht ein Block, der einen ReferenceError wirft:
```javascript
if (e.key === "Enter" || e.key === " ") {
e.preventDefault();
$(this).trigger("click");
}
```
`e` und `$(this)` sind hier nicht definiert.
## Lösung
- Doppelte `openLightbox()`/`closeLightbox()` Definitionen zusammenführen (nur Vanilla-Version behalten)
- Toten Code-Block entfernen
- Sicherstellen, dass alle Event-Listener korrekt funktionieren
## Akzeptanzkriterien
- [ ] Keine doppelten Funktionsdefinitionen mehr
- [ ] Keine ReferenceError in der Console
- [ ] Lightbox funktioniert weiterhin (Klick + Keyboard)
- [ ] Floor-Accordion funktioniert weiterhin
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Problem
Aus der Schwachstellenanalyse ergeben sich zwei zusammenhängende JS-Probleme in
js/haus-schleusingen.js:1. Doppelte Funktionsdefinitionen
openLightbox()undcloseLightbox()werden zweimal definiert – einmal mit jQuery, einmal vanilla. Die zweite Definition überschreibt die erste.2. Toter Code im Keyboard-Handler
Außerhalb jedes Event-Listeners steht ein Block, der einen ReferenceError wirft:
eund$(this)sind hier nicht definiert.Lösung
openLightbox()/closeLightbox()Definitionen zusammenführen (nur Vanilla-Version behalten)Akzeptanzkriterien