shuffle now applies to vertical slides as well

This commit is contained in:
Hakim El Hattab 2020-08-17 14:42:27 +02:00
parent 2bfe705e6a
commit 3a99a7b70a
4 changed files with 19 additions and 8 deletions

2
dist/reveal.esm.js vendored

File diff suppressed because one or more lines are too long

2
dist/reveal.js vendored

File diff suppressed because one or more lines are too long

View file

@ -16,8 +16,9 @@
<body>
<div class="reveal">
<div class="slides">
<section>Slide 1</section>
<section><section>Slide 1</section>
<section>Slide 2</section>
<section>Slide 3</section></section>
</div>
</div>

View file

@ -1467,13 +1467,23 @@ export default function( revealElement, options ) {
/**
* Randomly shuffles all slides in the deck.
*/
function shuffle() {
function shuffle( slides = getHorizontalSlides() ) {
getHorizontalSlides().forEach( ( slide, i, slides ) => {
slides.forEach( ( slide, i ) => {
// Insert this slide next to another random slide. This may
// cause the slide to insert before itself but that's fine.
dom.slides.insertBefore( slide, slides[ Math.floor( Math.random() * slides.length ) ] );
// Insert the slide next to a randomly picked sibling slide
// slide. This may cause the slide to insert before itself,
// but that's not an issue.
let beforeSlide = slides[ Math.floor( Math.random() * slides.length ) ];
if( beforeSlide.parentNode === slide.parentNode ) {
slide.parentNode.insertBefore( slide, beforeSlide );
}
// Randomize the order of vertical slides (if there are any)
let verticalSlides = slide.querySelectorAll( 'section' );
if( verticalSlides.length ) {
shuffle( verticalSlides );
}
} );