Skip to content

Conversation

@alexanbj
Copy link
Contributor

@alexanbj alexanbj commented Dec 18, 2025

Denne PRen skriver om Carousel-kompoenten til å bruke CSS scroll snap events, med fallback til intersection observer for de browserne som ikke støtter scroll snap events

Se https://developer.chrome.com/blog/scroll-snap-events og https://developer.mozilla.org/en-US/docs/Web/API/SnapEvent

Har også en annen branch klar med støtte for autoscroll basert på denne implementasjonen.

Endringer:

  • Endret change handleren til at den kun returnerer slide index, i stedet for et objekt med id og prevSlide index. Det er nå opp til konsument å håndtere dette selv.
  • Lagt til støtte for defaultInitialSlide prop
  • Skrevet om hele implementasjonen (med scroll snap events og intersection observer)
  • Fikset en bug i Firefox hvor piltastene ikke fungerte til å scrolle karusellen hvis ikke karusellen hadde blitt eksplisitt fokusert først
  • Fikset en bug hvor rask trykking på piltastene resulterete i en "halv scroll"
  • Knappene er nå disablet når det ikke kan scrolles lenger

@changeset-bot
Copy link

changeset-bot bot commented Dec 18, 2025

🦋 Changeset detected

Latest commit: 544b69f

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@obosbbl/grunnmuren-react Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@alexanbj alexanbj marked this pull request as ready for review January 6, 2026 09:05
@alexanbj alexanbj requested a review from a team as a code owner January 6, 2026 09:05
Copy link
Contributor

@oscarcarlstrom oscarcarlstrom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎠

@alexanbj alexanbj merged commit 53e1a2b into main Jan 6, 2026
3 checks passed
@alexanbj alexanbj deleted the carousel-v2 branch January 6, 2026 10:36
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.

4 participants