search

cyclestreets / cyclescape

Cyclescape - cycle campaign group toolkit
https://www.cyclescape.org/
MIT License
33 stars 15 forks source link

Latest thread position consistently failing in long thread #961

Closed mvl22 closed 4 years ago

mvl22 commented 4 years ago

In thread 832 I’m consistently being taken to this message (or possibly the one before, not sure):

/threads/832#message_71922

and not the latest message. This is a very long thread, so I wonder if there is something failing after a certain number of posted messages.

mvl22 commented 4 years ago

Thanks; this wasn't my imagination then!

mvl22 commented 4 years ago

What was wrong with the new method?

Incidentally, IE11 can now be the minimum supported browser - older IE versions are no longer supported by Microsoft.

nikolai-b commented 4 years ago

It changed how the scrolling animation and I feared that was a hot topic. I was prepared to do that when I thought it worked but moving from $(document).ready -> $(window).on('load', ...) is that actual fix so I went back to the old scrolling animation. Happy to use scrollIntoView if you think it won't upset anyone.

mvl22 commented 4 years ago

Ah good point; yes, let's not open that can of worms...