Skip to content

7538 feat(input-range): make _min and _max props required#8048

Closed
aTTiCuZ-CodingPage wants to merge 2 commits intopublic-ui:developfrom
aTTiCuZ-CodingPage:7538-Feature-input-range-min-and-max-as-required
Closed

7538 feat(input-range): make _min and _max props required#8048
aTTiCuZ-CodingPage wants to merge 2 commits intopublic-ui:developfrom
aTTiCuZ-CodingPage:7538-Feature-input-range-min-and-max-as-required

Conversation

@aTTiCuZ-CodingPage
Copy link
Copy Markdown
Contributor

@aTTiCuZ-CodingPage aTTiCuZ-CodingPage commented Jul 16, 2025

Refs: #7538

kol-input-range: Props _min und _max als required

Überblick

Dieses PR setzt die Migration von kol-input-range auf verpflichtende Props _min und _max um, im Rahmen von Kolibri v4. Die Änderung ist breaking, da bisher beide Props optional waren.

Änderungen im Detail

1. shadow.tsx

  • @Prop()-Deklarationen für _min und _max von optional (?) auf required (!) geändert.
  • Ergänzt um Initialwerte im internen State (zur Vermeidung von Laufzeitfehlern)

2. controller.ts

  • Bestehende Methoden validateMax() und validateMin() angepasst für required Props

3. schema/components/input-range.ts

  • _min und _max in:
    • RequiredProps (vorher optional)
    • RequiredStates (vorher optional) verschoben.

4. @else/all/component.tsx

  • Interne Sammelkomponente angepasst mit expliziten _min und _max Werten für <kol-input-range>

5. Tests (input-range.e2e.ts)

  • Für bestehende Test-Utilities ergänzt um additionalProperties: '_min="0" _max="100"'
  • Tests erfolgreich in allen Browsern (Chromium, Firefox, WebKit).

Breaking Change

Alle Implementierungen von <kol-input-range> müssen künftig _min und _max explizit übergeben.

Getestet

  • E2E-Tests (Chromium, Firefox, WebKit) erfolgreich.
  • Manuelle Tests in React-Samples
    • Ohne {...props} und ohne _min und _max = Fehlermeldung:
      • Type '{ _msg: { _type: "info"; _description: string; }; _label: string; }' is missing the following properties from type 'KolInputRange': "_max", "_min"

Additional Fixes

Während des build-and-check traten noch Fehler in den React-Samples "scenarios" auf.

Folgende Fixes habe ich durchgeführt:

  • Betroffene Dateien:

    • src/scenarios/disabled-interactive-elements.tsx
    • src/scenarios/focus-elements.tsx
    • src/scenarios/same-height-of-all-interactive-elements.tsx
    • src/scenarios/static-form.tsx
  • Änderung:
    Ich habe an den Stellen, wo <KolInputRange> verwendet wurde, die Props _min={0} und _max={100} ergänzt.

@anicyne anicyne linked an issue Jul 16, 2025 that may be closed by this pull request
@aTTiCuZ-CodingPage aTTiCuZ-CodingPage marked this pull request as draft July 16, 2025 12:32
@aTTiCuZ-CodingPage aTTiCuZ-CodingPage marked this pull request as ready for review July 16, 2025 14:36
@deleonio deleonio marked this pull request as draft July 22, 2025 00:42
@deleonio deleonio force-pushed the develop branch 5 times, most recently from f114c23 to dbfe961 Compare September 23, 2025 05:47
@deleonio deleonio closed this Oct 11, 2025
@publicuibot publicuibot bot locked and limited conversation to collaborators Oct 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Standardwerte für _min und _max beim InputRange

2 participants