You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In squiffy.ui.scrollToEnd, the scrolling is conditional on the content being lower than the current position. But when you switch to a new section with @clear, this works poorly, throwing the user to the page footer.
It seems to me the simplest fix would be to simply remove the if (scrollTo > maxScrollTop) and do scrolling no matter if we move up or down, but I might be missing some case where this leads to trouble.
The text was updated successfully, but these errors were encountered:
The if (scrollTo > maxScrollTop) just sets the maximum "scroll to" position as the end of the document, so it doesn't try to scroll beyond the end of the document. This is necessary for smooth scrolling animation, otherwise it will just suddenly stop once the scroll point reaches the maximum.
I haven't tested this yet but I expect the fix may be to set the scrollTop of the body and html elements to 0 when doing a clear.
In
squiffy.ui.scrollToEnd
, the scrolling is conditional on the content being lower than the current position. But when you switch to a new section with@clear
, this works poorly, throwing the user to the page footer.It seems to me the simplest fix would be to simply remove the
if (scrollTo > maxScrollTop)
and do scrolling no matter if we move up or down, but I might be missing some case where this leads to trouble.The text was updated successfully, but these errors were encountered: