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
As part of the development of our current auto advance system, code was added that keeps multiple auto advances at the same time from skipping over slides. It's better than nothing, but still broken.
Using display.previous_slide.auto_advance_duration in checking elapsed time seems wrong. Take for example four slides, which have durations of 2, 3, 2, and 2 seconds. With the current code, slides one, two, and three are displayed, but slide three's auto advance is ignored. We should probably use display.current_slide.auto_advance_duration, but it seems to me that there must have been some reason for doing it this way. More thinking is needed.
Even with this debounce code, we still sometimes have the same slide being triggered twice. This visually breaks transitions. We need to either debounce that event in _hyperscript, or debounce the view.
Debouncing the view seems possible once we implement a redis backend, but maybe a _hyperscript solution is in order in the interim.
The text was updated successfully, but these errors were encountered:
As part of the development of our current auto advance system, code was added that keeps multiple auto advances at the same time from skipping over slides. It's better than nothing, but still broken.
display.previous_slide.auto_advance_duration
in checking elapsed time seems wrong. Take for example four slides, which have durations of 2, 3, 2, and 2 seconds. With the current code, slides one, two, and three are displayed, but slide three's auto advance is ignored. We should probably usedisplay.current_slide.auto_advance_duration
, but it seems to me that there must have been some reason for doing it this way. More thinking is needed.Debouncing the view seems possible once we implement a redis backend, but maybe a _hyperscript solution is in order in the interim.
The text was updated successfully, but these errors were encountered: