Skip to content

Fix: Bildekvalitet-sliders viser og lagrer korrekte verdier (TT-2317)#96

Open
wrestlingcarbonapple wants to merge 3 commits into
masterfrom
feat/fix-image-size-settings-TT-2317
Open

Fix: Bildekvalitet-sliders viser og lagrer korrekte verdier (TT-2317)#96
wrestlingcarbonapple wants to merge 3 commits into
masterfrom
feat/fix-image-size-settings-TT-2317

Conversation

@wrestlingcarbonapple

Copy link
Copy Markdown
Contributor

Summary

  • Sliders for miniatyrbilder og forhåndsvisninger mangler value-prop og var ukontrollerte – de viste alltid default-posisjon uavhengig av lagret verdi
  • Forhåndsvisnings-slideren kalte setThumbnailSizeEdit i stedet for setPreviewSizeEdit, så forhåndsvisningsstørrelsen ble aldri oppdatert

Test plan

  • Åpne innstillinger og endre bildekvalitet for miniatyrbilder og/eller forhåndsvisninger
  • Verifiser at sliderne viser riktig posisjon etter at innstillingene er lastet
  • Lagre og lukk innstillinger, åpne på nytt – sliders skal vise lagrede verdier
  • Verifiser at miniatyrbilder og forhåndsvisninger kan endres uavhengig av hverandre

Closes TT-2317

🤖 Generated with Claude Code

wrestlingcarbonapple and others added 3 commits May 29, 2026 13:02
The build workflow bumps the version with `sed s/version = "0.0.0"/.../`,
but Cargo.toml had "0.0.1", so the pattern never matched and every build
shipped 0.0.1. VITE_APP_VERSION (the version gate's source) is baked from
Cargo.toml, so the gate flagged all builds as too old. Reset to 0.0.0 so
the existing sed matches again.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Legg til value-prop på begge sliders slik at de reflekterer lagret verdi
- Fiks at forhåndsvisnings-slider kalte setThumbnailSizeEdit i stedet for setPreviewSizeEdit

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

1 participant