From: Sven Schöling <s.schoeling@linet-services.de>
Date: Thu, 19 Jan 2017 13:18:57 +0000 (+0100)
Subject: Revert "focus: scroll to active element"
X-Git-Tag: release-3.5.4~1684
X-Git-Url: http://wagnertech.de/git?a=commitdiff_plain;h=6fc16dd1a7da25e9b728bdeb454a5e844850f809;p=kivitendo-erp.git

Revert "focus: scroll to active element"

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.
---

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;