Don't call iterate DOM several times in handlers
This commit is contained in:
parent
5a4fe7dbf9
commit
79f121259f
16
shower.js
16
shower.js
|
@ -42,7 +42,7 @@ window.shower = window.shower || (function(window, document, undefined) {
|
||||||
},
|
},
|
||||||
|
|
||||||
isLast : function() {
|
isLast : function() {
|
||||||
return shower.slideList.length === this.number + 1
|
return shower.slideList.length === this.number + 1;
|
||||||
},
|
},
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
@ -889,11 +889,13 @@ window.shower = window.shower || (function(window, document, undefined) {
|
||||||
shower.init();
|
shower.init();
|
||||||
|
|
||||||
document.addEventListener('click', function(e) {
|
document.addEventListener('click', function(e) {
|
||||||
var slideNumber = shower.getSlideNumber(shower._getSlideIdByEl(e.target)),
|
var slideId = shower._getSlideIdByEl(e.target),
|
||||||
|
slideNumber,
|
||||||
slide;
|
slide;
|
||||||
|
|
||||||
// Click on slide in List mode
|
// Click on slide in List mode
|
||||||
if (shower.isListMode() && shower._getSlideIdByEl(e.target)) {
|
if (slideId && shower.isListMode()) {
|
||||||
|
slideNumber = shower.getSlideNumber(slideId);
|
||||||
// Warning: go must be before enterSlideMode.
|
// Warning: go must be before enterSlideMode.
|
||||||
// Otherwise there is a bug in Chrome
|
// Otherwise there is a bug in Chrome
|
||||||
shower.go(slideNumber);
|
shower.go(slideNumber);
|
||||||
|
@ -908,11 +910,12 @@ window.shower = window.shower || (function(window, document, undefined) {
|
||||||
}, false);
|
}, false);
|
||||||
|
|
||||||
document.addEventListener('touchstart', function(e) {
|
document.addEventListener('touchstart', function(e) {
|
||||||
var slideNumber = shower.getSlideNumber(shower._getSlideIdByEl(e.target)),
|
var slideId = shower._getSlideIdByEl(e.target),
|
||||||
|
slideNumber,
|
||||||
slide,
|
slide,
|
||||||
x;
|
x;
|
||||||
|
|
||||||
if (shower._getSlideIdByEl(e.target)) {
|
if (slideId) {
|
||||||
if (shower.isSlideMode() && ! shower._checkInteractiveElement(e)) {
|
if (shower.isSlideMode() && ! shower._checkInteractiveElement(e)) {
|
||||||
x = e.touches[0].pageX;
|
x = e.touches[0].pageX;
|
||||||
|
|
||||||
|
@ -924,9 +927,10 @@ window.shower = window.shower || (function(window, document, undefined) {
|
||||||
}
|
}
|
||||||
|
|
||||||
if (shower.isListMode()) {
|
if (shower.isListMode()) {
|
||||||
|
slideNumber = shower.getSlideNumber(slideId);
|
||||||
// Warning: go must be before enterSlideMode.
|
// Warning: go must be before enterSlideMode.
|
||||||
// Otherwise there is a bug in Chrome
|
// Otherwise there is a bug in Chrome
|
||||||
shower.go(shower.getSlideNumber(shower._getSlideIdByEl(e.target)));
|
shower.go(slideNumber);
|
||||||
shower.enterSlideMode();
|
shower.enterSlideMode();
|
||||||
shower.showPresenterNotes(slideNumber);
|
shower.showPresenterNotes(slideNumber);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue