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
We've recently tried implementing the scroll view ability but are noticing an issue with the slide hash logic. Everything works as expected for the default horizontal navigation behaviour, but on scroll view it seems like any slides past the halfway point of the total slides seems to get offset by 1 index.
For example, here's with the default horizontal view, refreshing the page when the slide hash is present loads the correct slide as expected:
horizontal.mov
However, with scroll view enabled, when you have a slide more than halfway through the presentation in the hash, it's off by one index and loads the previous slide:
vertical.mov
Here's another example showing it happens on a presentation with less slides as well:
less_slides.mov
This is also an issue when using the slide API method. I tried debugging the reveal.js code and can't seem to find where this is happening - the variables seem like it has the correct slide but the index is getting altered somewhere along the way that I haven't tracked down. I also tried running a sync before calling slide but that also didn't seem to help.
Is there a way to get around this issue to make sure it uses the correct slide when hashes are used with scroll view? Thank you!
*xveganxxxedgex* created an issue (hakimel/reveal.js#3815)
<#3815>
Thank you for this library!
We've recently tried implementing the scroll view ability but are noticing
an issue with the slide hash logic. Everything works as expected for the
default horizontal navigation behaviour, but on scroll view it seems like
any slides past the halfway point of the total slides seems to get offset
by 1 index.
For example, here's with the default horizontal view, refreshing the page
when the slide hash is present loads the correct slide as expected:
https://github.com/user-attachments/assets/20943a26-f25b-429e-9c34-c55568c0c3fb
However, with scroll view enabled, when you have a slide more than halfway
through the presentation in the hash, it's off by one index and loads the
previous slide:
https://github.com/user-attachments/assets/ef5d42d7-5a8b-4db8-921a-9e20164dee2f
Here's another example showing it happens on a presentation with less
slides as well:
https://github.com/user-attachments/assets/28c70bfe-ab9d-404a-b92a-28d5bf9f3d30
This is also an issue when using the slide API method. I tried debugging
the reveal.js code and can't seem to find where this is happening - the
variables seem like it has the correct slide but the index is getting
altered somewhere along the way that I haven't tracked down. I also tried
running a sync before calling slide but that also didn't seem to help.
Is there a way to get around this issue to make sure it uses the correct
slide when hashes are used with scroll view? Thank you!
—
Reply to this email directly, view it on GitHub
<#3815>, or unsubscribe
<https://github.com/notifications/unsubscribe-auth/APGPZTRAU626XWGFZWMIGTL3O5IG7AVCNFSM6AAAAACESSGAP6VHI2DSMVQWIX3LMV43ASLTON2WKOZTGM2DMMJWGI2TGMQ>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
We don't need scroll view, so we disabled it in our project via scrollActivationWidth: 0. Not an answer to your question, and certainly not a real fix - but scroll view (in my opinion) isn't a very useful feature anyway, so for people stopping by because of a web search, this might be a useful hint.
Activity
hamed5962 commentedon Oct 21, 2025
fix(slides): disable scroll view on mobile to prevent crash (#10212)
jmuheim commentedon Sep 5, 2026
We experienced similar behavior.
We don't need scroll view, so we disabled it in our project via
scrollActivationWidth: 0. Not an answer to your question, and certainly not a real fix - but scroll view (in my opinion) isn't a very useful feature anyway, so for people stopping by because of a web search, this might be a useful hint.fix(slides): disable scroll view on mobile to prevent crash (marimo-t…
scroll mode: fix navigation landing on the previous slide
EvanCarroll commentedon Oct 16, 2026
That should fix it and it doesn't change the default.
The other patches changes the default and doesn't fix it.
Note: I pesonally think the default should change, but it should still be fixed.