Struppi: Review-Version des Lehrgangs

Beitrag lesen

die Review-Version (beta-Status) findest Du momentan hier:
http://aktuell.de.selfhtml.org/artikel/review/fader-framework/

Einige Anmerkungen.

Ein Konstruktor ist keine Fabrikmethode, im gegenteil eine Factory ersetzt den Kontruktor.

Bei den Beispielen solltest du deutlich unterscheidbarere Bilder nehmen, ich hab Schwierigkeiten den Effekt überhaupt wahrzunehmen.

Die Zeilen hat Mathias schon moniert:

                    if (this.images[i].style.filter != "undefined") {  
                        this.images[i].style.opacity = v/100;  
                        this.images[i].style.filter = "alpha(opacity=" + v + ")";  
                    }  

Das Skript funktioniert im FF nur, weil dein Test auf die filter Eigenschaft falsch ist. Wenn müßte es heißen:

  
if (typeof this.images[i].style.filter != "undefined")  
   this.images[i].style.filter = "alpha(opacity=" + v + ")";  
  
this.images[i].style.opacity = v/100;  

Aber du kannst die Prüfung auch komplett weglassen, die Browser stört es nicht, wenn du dem style Objekt eine unbekannte Eigenschaft zuweist.

Ich würde das FaderFramework Objekt mit new function() {...} erstellen, das hat den Vorteil, dass du in dem Konstruktor immer this bzw. deren Kopie verwenden kannst (falls du FaderFramework mal umbenennst, ist das hilfreich) und das du in der Funktion lokale Variabeln benutzen kannst.

Und eine klein Kleinigkeit:
oldWinOnLoad: false,
Das gegenteil von einer Funktionsreferenz ist meiner Meinung null, nicht false, kommt zwar auf's gleiche raus, aber mich stört sowas ;-)

Struppi.

3 52

Fader zum Bilder überblenden - ein kleiner Lehrgang

Felix Riesterer
  • javascript
  1. 2

    Das Script (Teil1)

    Felix Riesterer
    1. 1

      Das Script (Teil1) - Reflektion zum Code

      Felix Riesterer
      1. 0
        bosselmann
        1. 0
          Felix Riesterer
    2. 2

      alternativer loesungsansatz (teil1): model, view, ...

      peterS.
      1. 0

        alternativer loesungsansatz (teil2): .., controller.

        peterS.
        1. 0

          alternativer loesungsansatz (teil2): code review (teil 1?)

          peterS.
          1. 0

            alternativer loesungsansatz (teil2): code review (teil 2?)

            peterS.
            1. 0

              alternativer loesungsansatz (teil2): code review (teil 3)

              peterS.
      2. 0
        Felix Riesterer
        1. 0

          konkurrierende ansaetze

          peterS.
          1. 0
            Felix Riesterer
  2. 0

    SELFHTML-Artikel statt Forumsbeitrag?

    Vinzenz Mai
    • meinung
    1. 0
      Felix Riesterer
      1. 0
        Siechfred
        1. 0
          Felix Riesterer
        2. 0

          SELFHTML-Artikel - bin schon fleißig dabei!

          Felix Riesterer
        3. 0

          Artikel eingeschickt

          Felix Riesterer
          1. 0
            Siechfred
            • menschelei
            1. 0
              dedlfix
            2. 1
              Felix Riesterer
    2. 0
      Felix Riesterer
  3. 0

    Das Script (Teil 2)

    Felix Riesterer
  4. 0

    Das Script (Teil 3)

    Felix Riesterer
  5. 0

    Das Script (Endergebnis)

    Felix Riesterer
    1. 1
      molily
      1. 0
        Felix Riesterer
        1. 0
          Struppi
          1. 0
            Felix Riesterer
            1. 0
              bosselmann
              1. 0
                Felix Riesterer
                1. 0

                  Falscher Thread-Zweig...:-/

                  Felix Riesterer
                  1. 0

                    Verständnisfragen

                    bosselmann
                    1. 0
                      Felix Riesterer
                      1. 0
                        bosselmann
                        1. 0
                          Felix Riesterer
                          1. 0
                            bosselmann
                            1. 0
                              Felix Riesterer
                              1. 0

                                Skriptergänzung

                                bosselmann
                                1. 0
                                  Felix Riesterer
                                  1. 0

                                    Skriptergänzung (finale Version)

                                    bosselmann
                                    1. 0
                                      Felix Riesterer
  6. 0
    Felix Riesterer
    1. 0
      Beat
      1. 0

        Review-Version des Lehrgangs

        Felix Riesterer
        1. 0
          Beat
          1. 0
            Felix Riesterer
        2. 0
          Struppi
          1. 0
            Felix Riesterer
            1. 0
              Struppi
              1. 0
                Felix Riesterer