diff --git a/packages/page-controller/src/PageController.ts b/packages/page-controller/src/PageController.ts index 2439691..53b827d 100644 --- a/packages/page-controller/src/PageController.ts +++ b/packages/page-controller/src/PageController.ts @@ -322,11 +322,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, @@ -357,7 +357,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 8a86fa7..a218599 100644 --- a/packages/page-controller/src/actions.ts +++ b/packages/page-controller/src/actions.ts @@ -274,14 +274,7 @@ export async function scrollIntoViewIfNeeded(element: Element) { } } -/** - * @private Internal method, subject to change at any time. - */ -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 @@ -405,14 +398,7 @@ export async function scrollVertically( } } -/** - * @private Internal method, subject to change at any time. - */ -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 @@ -421,7 +407,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) @@ -469,7 +455,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 &&