Revision 6fc16dd1
Von Sven Schöling vor fast 8 Jahren hinzugefügt
| js/common.js | ||
|---|---|---|
| 235 | 235 |
} |
| 236 | 236 |
} |
| 237 | 237 |
|
| 238 |
// scroll to the active element. jQuery.focus doesn't do this |
|
| 239 |
// TODO: find out how to reliably frame the active element |
|
| 240 |
window.scrollTo(0, document.activeElement.getBoundingClientRect().top); |
|
| 241 |
|
|
| 242 | 238 |
// legacy. some forms install these |
| 243 | 239 |
if (typeof fokus == 'function') { fokus(); return; }
|
| 244 | 240 |
if (focus_by_name('cursor_fokus')) return;
|
Auch abrufbar als: Unified diff
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.