some refactoring...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
This commit is contained in:
Alex A. Naanou 2013-02-02 07:12:33 +04:00
parent 30083d52a2
commit 20a10ee3e4
3 changed files with 19 additions and 16 deletions

View File

@ -43,14 +43,6 @@ $(document).ready(function(){
// setup internal events... // setup internal events...
.on('pageChanged bookmarkAdded bookmarkRemoved bookmarksCleared', saveState) .on('pageChanged bookmarkAdded bookmarkRemoved bookmarksCleared', saveState)
.on('pageChanged', updatePageNumberIndicator) .on('pageChanged', updatePageNumberIndicator)
.on('bookmarksCleared', clearBookmarkIndicators)
.on('bookmarkAdded', function(_, n){makeBookmarkIndicator(n)})
.on('bookmarkRemoved', function(_, n){removeBookmarkIndicator(n)})
// editor specific events...
.on('pageCreated articleCreated magazineCreated', resetNavigator)
.on('pageMoved articleMoved', resetNavigator)
.on('pageRemoved articleRemoved', resetNavigator)
// templates... // templates...
.on('pageCreated articleCreated magazineCreated ' + .on('pageCreated articleCreated magazineCreated ' +

View File

@ -42,9 +42,6 @@ $(document).ready(function(){
// setup internal events... // setup internal events...
.on('pageChanged bookmarkAdded bookmarkRemoved bookmarksCleared', saveState) .on('pageChanged bookmarkAdded bookmarkRemoved bookmarksCleared', saveState)
.on('pageChanged', updatePageNumberIndicator) .on('pageChanged', updatePageNumberIndicator)
.on('bookmarksCleared', clearBookmarkIndicators)
.on('bookmarkAdded', function(_, n){makeBookmarkIndicator(n)})
.on('bookmarkRemoved', function(_, n){removeBookmarkIndicator(n)})
// user interactions... // user interactions...
.swipe({ .swipe({

View File

@ -45,7 +45,7 @@ function clearArticleIndicators(){
} }
function setupNavigator(){ function setupNavigator(skip_events){
var bar = $('.navigator .bar') var bar = $('.navigator .bar')
var elems = $('.navigator .indicator, .navigator .article') var elems = $('.navigator .indicator, .navigator .article')
var pos = $('.navigator .indicator').fadeIn() var pos = $('.navigator .indicator').fadeIn()
@ -68,10 +68,21 @@ function setupNavigator(){
makeBookmarkIndicator($(e).parents('.page')) makeBookmarkIndicator($(e).parents('.page'))
}) })
if(!skip_events){
// setup event handlers... // setup event handlers...
$('.viewer') $('.viewer')
// basic functions...
.on('pageChanged', function(e, n){updateNavigator(n)}) .on('pageChanged', function(e, n){updateNavigator(n)})
.on('magazineDragging', function(){updateNavigator()}) .on('magazineDragging', function(){updateNavigator()})
// bookmarks...
.on('bookmarksCleared', clearBookmarkIndicators)
.on('bookmarkAdded', function(_, n){makeBookmarkIndicator(n)})
.on('bookmarkRemoved', function(_, n){removeBookmarkIndicator(n)})
// editor specific events...
.on('pageCreated articleCreated magazineCreated', resetNavigator)
.on('pageMoved articleMoved', resetNavigator)
.on('pageRemoved articleRemoved', resetNavigator)
}
} }
// XXX this needs to unbind events... // XXX this needs to unbind events...
@ -118,6 +129,9 @@ function updateNavigator(n){
} }
/******************************************************* bookmarks ***/
function makeBookmarkIndicator(n){ function makeBookmarkIndicator(n){
if(n == null){ if(n == null){
n = getPageNumber() n = getPageNumber()