[bug] Alt+Q (scroll to anchor) op forum werkt niet meer?

[RayNbow]

Een van de features op GoT is het snel kunnen springen naar de gelinkte post m.b.v. Alt+Q (de standaard hotkey). Dit werkte nog in de oude Edge browser, maar in de nieuwe Edge gebaseerd op Chromium werkt deze feature niet meer. In Chrome zelf werkt de feature ook niet. In plaats van naar de gelinkte post te springen, springt de browser naar het begin van de pagina.

[DaFeliX]

Hm, bij mij in Firefox werken ze wel (Alt+shift), maar volgens mij zou het in Edge met Alt wel moeten werken. Werken Alt+w (focus quickreply) en Alt+f (zoeken) wel? Heb je toevallig de instellingen aangepast in je profiel?

[RayNbow]

DaFeliX schreef op donderdag 2 april 2020 @ 19:46:
Hm, bij mij in Firefox werken ze wel (Alt+shift),

Net getest in Firefox; Alt+Shift+Q werkt.

maar volgens mij zou het in Edge met Alt wel moeten werken.

In de oude Edge werkte het, maar in Edge Chromium (Version 80.0.361.109 (Official build) (64-bit)) niet.

Werken Alt+w (focus quickreply) en Alt+f (zoeken) wel?

Ja, die werken wel.

Heb je toevallig de instellingen aangepast in je profiel?

Nee, die instellingen heb ik niet aangepast.


Het punt is dat Edge Chromium wel reageert op Alt+Q, maar dat de scrollbarpositie na het indrukken op 0 wordt gezet. Heb je misschien wat tips waar ik in de code misschien een breakpoint kan zetten om te kijken wat er precies gebeurt als ik op Alt+Q druk?

[DaFeliX]

RayNbow schreef op donderdag 2 april 2020 @ 20:06:
[...]

Nee, die instellingen heb ik niet aangepast.


Het punt is dat Edge Chromium wel reageert op Alt+Q, maar dat de scrollbarpositie na het indrukken op 0 wordt gezet. Heb je misschien wat tips waar ik in de code misschien een breakpoint kan zetten om te kijken wat er precies gebeurt als ik op Alt+Q druk?

Het is een minified script (forum.js, methode "scrolltoanchor"). Als je "scrolltoanchor()" in je console (F12 gebruiken om devtools te openen) uitvoert, kom je dan wel op de juiste positie terecht of stuurt die je ook naar scrollpositie 0?

[crisp]

Dit lijkt inderdaad niet meer te werken in chrome:

JavaScript:

function scrolltoanchor(h)
{
    if (!h) h = window.location.hash;
    if (h != '') window.location.hash = h;
}

Het is dan ook wel erg oude code (en in de codebase al minstens 12 jaar niet meer aangeraakt).

Ik denk dat voor deze specifieke usecase ik het zou herschrijven naar zoiets:

JavaScript:

function scrolltoanchor()
{
    var messageId = window.location.hash.substr(1),
        anchor    = messageId && document.getElementsByName(messageId).item(0);

    if (anchor)
    {
        anchor.scrollIntoView();
    }
}

[crisp]

Ik heb hier lokaal een fix voor gemaakt

[RayNbow]

Als tijdelijke oplossing heb ik bovenstaande scrolltoanchor definitie opgenomen in een simpel Tampermonkey script.