From 6726e5b0dcbd40a9ef8cf30d4ec5651524d4d2cd Mon Sep 17 00:00:00 2001 From: Hidde Jansen Date: Sun, 6 Nov 2011 19:40:51 +0100 Subject: [PATCH] Scratched some todos off the list --- lib/ComicBook.js | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/lib/ComicBook.js b/lib/ComicBook.js index 21fc023..54a838d 100755 --- a/lib/ComicBook.js +++ b/lib/ComicBook.js @@ -78,7 +78,8 @@ function ComicBook(id, srcs, opts) { previous: 80, toolbar: 84, toggleLayout: 76 - } + }, + forward_buffer: 3 }; var options = merge(defaults, opts); // options array for internal use @@ -169,11 +170,9 @@ function ComicBook(id, srcs, opts) { } // add page controls - // TODO: add IE event listeners too. canvas.addEventListener("click", ComicBook.prototype.navigation, false); window.addEventListener("keydown", ComicBook.prototype.navigation, false); - window.addEventListener("hashchange", checkHash, false); - //setInterval(function() { checkHash(); }, 300); // TODO: enable this when there is no onhashchange event + $(window).bind('hashchange', checkHash {}); } /** @@ -526,9 +525,6 @@ function ComicBook(id, srcs, opts) { // loads pages in both directions so you don't have to wait for all pages // to be loaded before you can scroll backwards function preload(start, stop) { - - var forward_buffer = 3; // TODO: make this into a config option? - var j = 0; var count = 1; var forward = start; @@ -536,7 +532,7 @@ function ComicBook(id, srcs, opts) { while (forward <= stop) { - if (count > forward_buffer && backward > -1) { + if (count > options.forward_buffer && backward > -1) { queue.push(backward); backward--; count = 0;