Skip to content

Prevent warning - opening CP or vehicle card from table#2642

Closed
Brett-S-OWB wants to merge 2 commits intoopenWB:masterfrom
Brett-S-OWB:inject-table-warn
Closed

Prevent warning - opening CP or vehicle card from table#2642
Brett-S-OWB wants to merge 2 commits intoopenWB:masterfrom
Brett-S-OWB:inject-table-warn

Conversation

@Brett-S-OWB
Copy link
Contributor

Um die Kartenbreite zu berechnen und in der Basiskarussell-Komponente die Anzahl der anzuzeigenden Karten zu bestimmen, wird die Funktion setCardWidth in ChargePointCard und VehicleCard injiziert. Wenn jedoch das Karussell nicht verwendet wird und stattdessen die Tabellenansicht aktiv ist, wird eine Warnung ausgegeben, da die Funktion vom Karussell nicht übergeben wurde. Aus diesem Grund wird hier ein Fallback auf undefined verwendet, der jedoch in denselben Funktionstyp wie der von BaseCarousel erwartet wird umgewandelt werden muss, um TypeScript-Fehler zu vermeiden.

@benderl benderl mentioned this pull request Aug 14, 2025
@benderl
Copy link
Contributor

benderl commented Aug 14, 2025

Das ganze Konstrukt mit der Berechnung der Gruppengröße ist sehr unhandlich geworden. Mit 0bada3c in #2662 habe ich das überarbeitet. Die komplette Logik ist jetzt in BaseCarousel enthalten und es konnte viel Code entfernt werden.

@benderl benderl closed this Aug 14, 2025
@Brett-S-OWB Brett-S-OWB deleted the inject-table-warn branch August 15, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants