diff --git a/packages/page-controller/src/PageController.ts b/packages/page-controller/src/PageController.ts index 2f56a73..ac83b15 100644 --- a/packages/page-controller/src/PageController.ts +++ b/packages/page-controller/src/PageController.ts @@ -321,11 +321,11 @@ export class PageController extends EventTarget { this.assertIndexed() - const scrollAmount = pixels ?? numPages * (down ? 1 : -1) * window.innerHeight + const scrollAmount = (pixels ?? numPages * window.innerHeight) * (down ? 1 : -1) const element = index !== undefined ? getElementByIndex(this.selectorMap, index) : null - const message = await scrollVertically(down, scrollAmount, element) + const message = await scrollVertically(scrollAmount, element) return { success: true, @@ -356,7 +356,7 @@ export class PageController extends EventTarget { const element = index !== undefined ? getElementByIndex(this.selectorMap, index) : null - const message = await scrollHorizontally(right, scrollAmount, element) + const message = await scrollHorizontally(scrollAmount, element) return { success: true, diff --git a/packages/page-controller/src/actions.ts b/packages/page-controller/src/actions.ts index 92bb2b4..dcf6ab4 100644 --- a/packages/page-controller/src/actions.ts +++ b/packages/page-controller/src/actions.ts @@ -231,11 +231,7 @@ export async function scrollIntoViewIfNeeded(element: Element) { } } -export async function scrollVertically( - down: boolean, - scroll_amount: number, - element?: HTMLElement | null -) { +export async function scrollVertically(scroll_amount: number, element?: HTMLElement | null) { // Element-specific scrolling if element is provided if (element) { const targetElement = element @@ -359,11 +355,7 @@ export async function scrollVertically( } } -export async function scrollHorizontally( - right: boolean, - scroll_amount: number, - element?: HTMLElement | null -) { +export async function scrollHorizontally(scroll_amount: number, element?: HTMLElement | null) { // Element-specific scrolling if element is provided if (element) { const targetElement = element @@ -372,7 +364,7 @@ export async function scrollHorizontally( let scrolledElement: HTMLElement | null = null let scrollDelta = 0 let attempts = 0 - const dx = right ? scroll_amount : -scroll_amount + const dx = scroll_amount while (currentElement && attempts < 10) { const computedStyle = window.getComputedStyle(currentElement) @@ -420,7 +412,7 @@ export async function scrollHorizontally( // Page-level scrolling (default or fallback) - const dx = right ? scroll_amount : -scroll_amount + const dx = scroll_amount const bigEnough = (el: HTMLElement) => el.clientWidth >= window.innerWidth * 0.5 const canScroll = (el: HTMLElement | null) => el &&