[END] key bug

[lokpro]

Pressing the [END] key should go to the last slide, but the current code doesn't work.
It won't go to my last slide (11) but to number 4. It seems like the number param is taken modulus to be the slide number.

reveal.js/js/controllers/keyboard.js

Line 338 in 90bbe8b

this.Reveal.slide( Number.MAX_VALUE );

shift+➜ too,

reveal.js/js/controllers/keyboard.js

Line 299 in 90bbe8b

this.Reveal.slide( Number.MAX_VALUE );

should be something like

Reveal.slide( Reveal.getSlides().length-1 );

[hakimel]

Is this in a presentation with vertical slides? Any chance you could share a link to it? I tried out Reveal.slide( Number.MAX_VALUE ) but it took me to the correct slide in the presentations I used.

[lokpro]

I found out that with loop=true, numbers larger than total slides will jump back around. That's why MAX_VALUE is not going to the last slide.

[lokpro]

Reveal.slide(
   lastIndexOfHSlides, 
   lastIndexOfVSlides,
   lastIndexOfFragments,
);

might be something like this.

// or Just an idea, a 3-step [END], [END], [END]

Reveal.slide(
   lastIndexOfHSlides,
   isAlreadyOnLastHSlide ? lastIndexOfVSlides : 0,
   isAlreadyOnLastHSlide && isAlreadyOnLastVSlide ? lastIndexOfFragments : 0,
);

[hakimel]

This has been fixed now. Shortcuts for jumping to the last slide now work correctly even if the presentation is looped.