From 6fc16dd1a7da25e9b728bdeb454a5e844850f809 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Sven=20Sch=C3=B6ling?= Date: Thu, 19 Jan 2017 14:18:57 +0100 Subject: [PATCH] Revert "focus: scroll to active element" MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit This reverts commit fdc625b4d1ba26782ff81ec2d79c1b533f26a9a7. Das funktioniert so nicht. Grundsätzlich passiert das Scrollen passiert bei Fokus automatisch. Bei force-reload kann es aber passieren, dass js/common ausgeführt wird, bevor alle css Dateien geladen sind. Dann ändert sich nachträglich die Größe der Elemente und das Element wird wieder aus dem Bildschirm geschoben. Ausserdem können Browser kein scroll offset, focus scrollt nur exakt soweit dass das Element auf dem Screen ist, auch wenn das bedeutet, dass es dann in den letzten 3% des Screen liegt. Ich habe versucht das zu fixen, aber es wird dadurch nur komplizierter, nicht besser. Also raus damit. --- js/common.js | 4 ---- 1 file changed, 4 deletions(-) diff --git a/js/common.js b/js/common.js index f8533d5e7..285b02fef 100644 --- a/js/common.js +++ b/js/common.js @@ -235,10 +235,6 @@ $(function () { } } - // scroll to the active element. jQuery.focus doesn't do this - // TODO: find out how to reliably frame the active element - window.scrollTo(0, document.activeElement.getBoundingClientRect().top); - // legacy. some forms install these if (typeof fokus == 'function') { fokus(); return; } if (focus_by_name('cursor_fokus')) return; -- 2.20.1