bugfix: carousel.js (#325)

rol4nd909 · web-flow · commit 82317d0d04c4 · 2025-02-19T10:50:50.000-08:00
this solves an issue where on large screens the goToElement is skipping items.
diff --git a/carousel/carousel.js b/carousel/carousel.js
@@ -98,9 +98,7 @@ export default class Carousel {
     const delta = Math.abs(scrollport.offsetLeft - element.offsetLeft)
     const scrollerPadding = parseInt(getComputedStyle(scrollport)['padding-left'])
 
-    const pos = scrollport.clientWidth / 2 > delta
-      ? delta - scrollerPadding
-      : delta + scrollerPadding
+    const pos = delta - scrollerPadding
 
     scrollport.scrollTo(dir === 'ltr' ? pos : pos*-1, 0)
   }

Original file line number	Diff line number	Diff line change
`@@ -98,9 +98,7 @@ export default class Carousel {`
`98`	`98`	`const delta = Math.abs(scrollport.offsetLeft - element.offsetLeft)`
`99`	`99`	`const scrollerPadding = parseInt(getComputedStyle(scrollport)['padding-left'])`
`100`	`100`
`101`		`- const pos = scrollport.clientWidth / 2 > delta`
`102`		`- ? delta - scrollerPadding`
`103`		`- : delta + scrollerPadding`
	`101`	`+ const pos = delta - scrollerPadding`
`104`	`102`
`105`	`103`	`scrollport.scrollTo(dir === 'ltr' ? pos : pos*-1, 0)`
`106`	`104`	`}`