From 22bc09dc8be66cb104009043ff370ad3b5abf794 Mon Sep 17 00:00:00 2001 From: Marc Durdin Date: Fri, 4 Sep 2026 16:11:05 +0200 Subject: [PATCH] fix(web): remove scroll-to-top on page load This is a legacy behaviour from pre KeymanWeb 2.0 days, probably to deal with buggy rendering in older browsers. Removing as an unwanted side-effect. Fixes: #16503 Test-bot: skip --- .../browser/src/context/pageIntegrationHandlers.ts | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/web/src/app/browser/src/context/pageIntegrationHandlers.ts b/web/src/app/browser/src/context/pageIntegrationHandlers.ts index 8c2931731d6..a17755d9ead 100644 --- a/web/src/app/browser/src/context/pageIntegrationHandlers.ts +++ b/web/src/app/browser/src/context/pageIntegrationHandlers.ts @@ -159,15 +159,6 @@ export class PageIntegrationHandlers { return false; }; - - private _WindowLoad: (e: Event) => void = () => { - // Always return to top of page after a page reload - document.body.scrollTop=0; - if(typeof document.documentElement != 'undefined') { - document.documentElement.scrollTop=0; - } - } - private attachHandlers() { const eventTracker = this.domEventTracker; const device = this.engine.config.hostDevice; @@ -191,8 +182,6 @@ export class PageIntegrationHandlers { eventTracker.attachDOMEvent(docBody, 'touchend', this.touchEndActivationHandler, false); } - eventTracker.attachDOMEvent(window, 'load', this._WindowLoad, false); - eventTracker.attachDOMEvent(document, 'keyup', this.engine.hotkeyManager._Process, false); } @@ -217,8 +206,6 @@ export class PageIntegrationHandlers { this.mobilePageTrailer?.parentElement.removeChild(this.mobilePageTrailer); } - eventTracker.detachDOMEvent(window, 'load', this._WindowLoad, false); - eventTracker.detachDOMEvent(document, 'keyup', this.engine.hotkeyManager._Process, false); } } \ No newline at end of file