Scroll to Top bij sorteren op Totaalprijs?

[Olaf van der Spek]

pricewatch: Samsung 850 EVO 250GB#

[rens-br]

Olaf van der Spek schreef op woensdag 25 januari 2017 @ 13:28:
pricewatch: Samsung 850 EVO 250GB#

Wat bedoel je precies? Zou je iets meer informatie willen geven, want op deze manier kunnen we er helaas weinig mee.

[Olaf van der Spek]

Scroll naar beneden en klik een paar keer op Totaalprijs.

[donny007]

Ik denk dat dit het (ongewenste) gedrag van de pagina is waar Olaf op doelt:



Bij sorteren op totaalprijs scrolt de pagina (bij de tweede keer klikken) naar boven.

Browser: Firefox 51.0
OS: Windows 10 x64

[crisp]

Dat is browser-behaviour; een leeg fragment zorgt hiervoor. Dat is helaas niet te voorkomen...

[Olaf van der Spek]

crisp schreef op dinsdag 31 januari 2017 @ 09:59:
Dat is helaas niet te voorkomen...

Dat kan toch niet waar zijn? Sowieso gebeurd het niet bij elke klik..
Kan het fragment niet verwijderd worden?

[crisp]

Olaf van der Spek schreef op dinsdag 31 januari 2017 @ 10:24:
[...]

Dat kan toch niet waar zijn? Sowieso gebeurd het niet bij elke klik..
Kan het fragment niet verwijderd worden?

Wij gebruiken de fragment om de 'state' bij te houden aangezien de interactie via ajax verloopt. Dat maakt de pagina ook bookmarkable. Wanneer je bij sorteren weer teruggaat naar de 'default' state dan is er feitelijk geen informatie die in de fragment hoeft; technisch wordt er dan location.hash='' gedaan. De enige manier om dan scrolling te voorkomen is om dan maar een 'talisman' te gebruiken, maar imo is dat wel een ranzige oplossing voor een probleem dat maar relatief weinig voorkomt en weinig impact heeft.

[Room42]

return false;

[crisp]

Room42 schreef op dinsdag 31 januari 2017 @ 11:15:
return false;

Nee, dat heeft geen nut

Er is nog wel een mogelijkheid in moderne browsers om het via de history API op te lossen, maar nogmaals: het is imo een onbeduidend probleem...

[Room42]

Ik weet niet wat een talisman is, maar kun je dan inderdaad geen '#default' gebruiken of zo
Ik snap wel dat het een minor issue is, maar wel lelijk

[crisp]

Ik kan er wel #care van maken

[Room42]

Haha! Dat lijkt me het beste, ja

[crisp]

Ik heb het lokaal opgelost met pushState; over 2 weken online