Verbesserung für JavaScript scrollToAnchor() im Backend

[aeberhard]

Da beim speichern von Content-Slices (zumindest bei mir) sehr oft nicht korrekt zum Anfang des Slices gesprungen wurde habe ich mal nach Besserung gesucht. Liegt evtl. an Bildern die im Backend angezeigt werden und erst geladen werden, oder an Tabs die ich in Modul-Inputs verwende.

Folgende Anpassung ergibt bei mir ein wesentlich besseres Verhalten beim speichern von Slices. ;-)
Ganz ohne Berechnungen ...

scrollToAnchor = function() {
    if (window.location.hash) {
            var anchorItem = document.querySelector(window.location.hash);
            if (anchorItem) {
                anchorItem.scrollIntoView({ behavior: "smooth", block: "start"});
            }
    }
};

[aeberhard]

Hier noch eine verbesserte Version da abhängig von der aktuellen Scroll-Position wenn Speichern/Übernehmen geklickt wird
der Scroll noch nicht exakt funktionierte.

$(document).on('pjax:end', function() {
    scrollToAnchor();
});

scrollToAnchor = function() {
    if (window.location.hash) {
        var hash = window.location.hash;
        var anchorItem = document.querySelector(hash);

        if (anchorItem) {
            // Ein minimaler Timeout (0-10ms) hilft oft Wunder bei Pjax-Updates
            setTimeout(function() {
                anchorItem.scrollIntoView({ 
                    block: "start", 
                    behavior: "smooth" 
                });
            }, 50);
        }
    }
};

[gharlan]

Mir fällt es schwer, das zu bewerten/einzuschätzen. Aber als PR ließe sich das zumindest schon mal besser vergleichen, was genau geändert werden soll.

[ynamite]

Bei meinen ersten Tests scheint das tatsächlich besser zu funktionieren. Ich teste noch weiter.