Template ce_noobslide_end

Hallo Philipp,

die Ausgabe des Templates ce_noobslide_end könnte etwas einfacher aussehen
vorher

HTML-Code:

---

var controlDiv = new Element('div', {'class':'control_buttons'}).inject(document.getElement('#kaipoNoobSlider'));
for (var i = 1; i <= 7; i++){new Element('span', {'class':'ce_noobSlide_button ce_noobSlide_controls', html:i}).inject(document.getElement('#kaipoNoobSlider .control_buttons'));}
var noobslide71;
window.addEvent('domready', function() {
  noobslide71 = new noobSlide({
    box: document.getElement('#kaipoNoobSlider .ce_noobSlide_container'),
    items: [0,1,2,3,4,5,6],
    mode:'horizontal',
    size: 950,
      interval: 8000,
      fadeDuration: 1500,
      autoPlay: 1
    ,
    onWalk: function(currentItem,currentHandle){
        this.handles.removeClass('active');
        currentHandle.addClass('active');
      }
    ,
    handles: $$('#kaipoNoobSlider .ce_noobSlide_controls')
  });

new MooSwipe(document.getElement('#kaipoNoobSlider .ce_noobSlide_container'), {
    onSwipeLeft: function() {
      noobslide71.next(this);
      },
    onSwipeRight: function() {
      noobslide71.previous(this);
    }
  });

});

---

nachher

HTML-Code:

---

window.addEvent('domready', function() {

  var controlDiv = new Element('div.control_buttons').inject('kaipoNoobSlider');
  for (var i = 1; i <= 7; i++){
    new Element('span', {
      class: 'ce_noobSlide_button ce_noobSlide_controls', // warum 2 Klassen?
      text: i
    }).inject(controlDiv);
  }

  var noobslide71;

  noobslide71 = new noobSlide({
    box: $$('#kaipoNoobSlider .ce_noobSlide_container')[0],
    items: [0,1,2,3,4,5,6],
    mode: 'horizontal',
    size: 950,
    interval: 8000,
    fadeDuration: 1500,
    autoPlay: 1,
    onWalk: function(currentItem, currentHandle){
        this.handles.removeClass('active');
        currentHandle.addClass('active');
      },
    handles: $$('#kaipoNoobSlider .ce_noobSlide_controls')
  });

  new MooSwipe($$('#kaipoNoobSlider .ce_noobSlide_container')[0], {
    onSwipeLeft: function() {
      noobslide71.next(this);
    },
    onSwipeRight: function() {
      noobslide71.previous(this);
    }
  });

});

---

Hauptsächlich sollte der oberste Teil schon mit in den domready, um sicher zu stellen, dass das Element vorhanden ist. Zweitens benutze ich die Elemente, die erstellt wurden, anstatt eine neue Suche danach zu starten.