janishutz-legacy/impress.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

impressive refactoring - no need to search for active step every time...

Browse Source
This commit is contained in:
Bartek Szopka
2012-01-04 22:42:11 +01:00
parent 09a021ec38
commit e7b8b9b26c
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
js/impress.js
View File

@@ -184,6 +184,8 @@
// making given step active // making given step active
var active = null;
var select = function ( el ) { var select = function ( el ) {
if ( !el || !el.stepData ) { if ( !el || !el.stepData ) {
// selected element is not defined as step // selected element is not defined as step
@@ -202,8 +204,8 @@
var step = el.stepData; var step = el.stepData;
if ( $(".step.active", impress) ) { if ( active ) {
$(".step.active", impress).classList.remove("active"); active.classList.remove("active");
} }
el.classList.add("active"); el.classList.add("active");
@@ -247,6 +249,7 @@
}); });
current = target; current = target;
active = el;
return el; return el;
} }
@@ -255,7 +258,6 @@
document.addEventListener("keydown", function ( event ) { document.addEventListener("keydown", function ( event ) {
if ( event.keyCode == 9 || ( event.keyCode >= 32 && event.keyCode <= 34 ) || (event.keyCode >= 37 && event.keyCode <= 40) ) { if ( event.keyCode == 9 || ( event.keyCode >= 32 && event.keyCode <= 34 ) || (event.keyCode >= 37 && event.keyCode <= 40) ) {
var active = $(".step.active", impress);
var next = active; var next = active;
switch( event.keyCode ) { switch( event.keyCode ) {
case 33: ; // pg up case 33: ; // pg up