Mise au point.

[Portfolio.git] / skins / photo_lightbox_viewer.js

diff --git a/skins/photo_lightbox_viewer.js b/skins/photo_lightbox_viewer.js
index d659ac1..a4c9a25 100644 (file)
--- a/skins/photo_lightbox_viewer.js
+++ b/skins/photo_lightbox_viewer.js
@@ -14,10 +14,9 @@ var reSelected = /.*selected.*/;
 Lightbox = function(grid, toolbar, complete) {
        var self = this;
        this.grid = grid;
-       this.lastSlide = this.grid.children[this.grid.children.length-1];
+       this._buildSlidesIndex(); // set this.slides and this.lastSlide;
        this.fetchingDisabled = false;
        this.complete = complete;
-       console.log('complete:', complete)
        this.toolbar = toolbar;
        if (toolbar) {
                this.toolbarFixed = false;
@@ -47,6 +46,18 @@ Lightbox = function(grid, toolbar, complete) {
        }
 };
 
+Lightbox.prototype._buildSlidesIndex = function() {
+       this.slides = [];
+       var node, i;
+       for (i=0 ; i<this.grid.childNodes.length ; i++) {
+               node = this.grid.childNodes[i];
+               if (node.nodeType === 1) { // is element
+                       this.slides.push(node);
+               }
+       }
+       this.lastSlide = this.slides[this.slides.length-1];
+};
+
 Lightbox.prototype.windowScrollToolbarlHandler = function(evt) {
        if (this.toolbar.offsetTop < window.scrollY && !this.toolbarFixed) {
                this.toolbarFixed = true;
@@ -247,7 +258,7 @@ Lightbox.prototype.deleteSelection = function() {
 };
 
 Lightbox.prototype._removeSelection = function() {
-       var i, e, slide;
+       var i, e;
        var toRemove = [];
        for (i=0 ; i<this.form.elements.length ; i++) {
                e = this.form.elements[i];
@@ -256,10 +267,11 @@ Lightbox.prototype._removeSelection = function() {
                }
        }
        for (i=0 ; i<toRemove.length ; i++) {
-               slide = toRemove[i];
-               slide.parentNode.removeChild(slide);
+               this.grid.removeChild(toRemove[i]);
        }
+       this._buildSlidesIndex();
        this.cbIndex = undefined;
+       this.windowScrollGridHandler();
 };
 
 Lightbox.prototype.getCBIndex = function(cb) {
@@ -267,14 +279,11 @@ Lightbox.prototype.getCBIndex = function(cb) {
                // build checkbox index
                this.cbIndex = [];
                var i, node, c;
-               var nodes = this.grid.childNodes;
-               for (i=0 ; i<nodes.length ; i++) {
-                       node = nodes[i];
-                       if (node.nodeName === 'SPAN') {
-                               c = node.getElementsByTagName('input')[0];
-                               c.index = this.cbIndex.length;
-                               this.cbIndex[this.cbIndex.length] = c;
-                       }
+               for (i=0 ; i<this.slides.length ; i++) {
+                       node = this.slides[i];
+                       c = node.getElementsByTagName('input')[0];
+                       c.index = this.cbIndex.length;
+                       this.cbIndex.push(c);
                }
        }
        return cb.index;
@@ -321,18 +330,21 @@ Lightbox.prototype.refreshGrid = function() {
        
        var url = absolute_url() +
                          '/portfolio_thumbnails_tail?start:int=0&size:int=' +
-                         this.grid.children.length;
+                         this.slides.length;
        req.open('GET', url, true);
        req.send();
 };
 
 Lightbox.prototype._refreshGrid = function(req) {
        var doc = req.responseXML.documentElement;
-       var i;
-       var slides = this.grid.children;
-       for (i=0 ; i<doc.children.length ; i++) {
-               this.grid.replaceChild(getCopyOfNode(doc.children[i]), slides[i]);
+       var i, node;
+       for (i=0 ; i<doc.childNodes.length ; i++) {
+               node = doc.childNodes[i];
+               if (node.nodeType === 1) {
+                       this.slides[i] = this.grid.replaceChild(getCopyOfNode(node), this.slides[i]);
+               }
        }
+       this.cbIndex = undefined;
 };
 
 Lightbox.prototype.fetchTail = function() {
@@ -354,7 +366,7 @@ Lightbox.prototype.fetchTail = function() {
        
        var url = absolute_url() +
                          '/portfolio_thumbnails_tail?start:int=' +
-                         String(this.grid.children.length + 1 ) +
+                         String(this.slides.length + 1 ) +
                          '&size:int=10';
        req.open('GET', url, true);
        req.send();
@@ -362,15 +374,23 @@ Lightbox.prototype.fetchTail = function() {
 
 Lightbox.prototype._appendTail = function(req) {
        var doc = req.responseXML.documentElement;
-       var i;
-       var slides = this.grid.children;
-       for (i=0 ; i<doc.children.length ; i++) {
-               this.lastSlide = this.grid.appendChild(getCopyOfNode(doc.children[i]));
+       var i, node, c;
+       for (i=0 ; i<doc.childNodes.length ; i++) {
+               node = doc.childNodes[i];
+               if (node.nodeType === 1) {
+                       this.lastSlide = this.grid.appendChild(getCopyOfNode(node));
+                       this.slides.push(this.lastSlide);
+                       if (this.cbIndex) {
+                               c = this.lastSlide.getElementsByTagName('input')[0];
+                               c.index = this.cbIndex.length;
+                               this.cbIndex.push(c);
+                               
+                       }
+               }
        }
        this.fetchingDisabled = false;
        if (doc.getAttribute('nomore')) {
                this.complete = true;
-               console.info('complete');
        }
        this.windowScrollGridHandler();
 };