From aeb5a85cfd1485be3374a730a6a88f9e82b36ad6 Mon Sep 17 00:00:00 2001
From: "Alex A. Naanou" <alex.nanou@gmail.com>
Date: Tue, 29 Jan 2013 02:28:10 +0400
Subject: [PATCH] some cleanup...

Signed-off-by: Alex A. Naanou <alex.nanou@gmail.com>
---
 magazine.js | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/magazine.js b/magazine.js
index abb467b..7f44bb8 100755
--- a/magazine.js
+++ b/magazine.js
@@ -651,8 +651,7 @@ function updateNavigator(n){
 }
 
 
-// XXX store bookmarks in a seporate place...
-// NOTE: this depends on the indicator bar...
+
 function makeBookmarkIndicator(n){
 	if(n == null){
 		n = getPageNumber()
@@ -685,16 +684,16 @@ function removeBookmarkIndicator(n){
 	$('.navigator .bar .bookmark[page="'+n+'"]').remove()
 }
 
-// XXX move to actions...
+
+
+/******************************************************* bookmarks ***/
+
+// load bookmarks from list...
 function loadBookmarks(lst){
 	clearBookmarks()
 	$(lst).each(function(i, e){toggleBookmark(e)})
 }
-function clearBookmarks(){
-	$('.magazine .page .bookmark').remove()
-	clearBookmarkIndicators()
-}
-
+// build bookmark list...
 function buildBookmarkList(){
 	var res = []
 	$('.magazine .page .bookmark').each(function(_, e){
@@ -703,6 +702,12 @@ function buildBookmarkList(){
 	return res
 }
 
+function clearBookmarks(){
+	$('.magazine .page .bookmark').remove()
+	clearBookmarkIndicators()
+}
+
+
 function toggleBookmark(n){
 	if(n == null){
 		n = getPageNumber()
@@ -732,8 +737,6 @@ function toggleBookmark(n){
 	return res
 }
 
-
-// XXX move this to actions...
 function nextBookmark(){
 	var pages = $('.page')
 	pages = $(pages.splice(getPageNumber()+1))
@@ -752,6 +755,7 @@ function prevBookmark(){
 }
 
 
+
 /********************************************************** editor ***/
 
 // XXX create magazine...