• Jetzt anmelden. Es dauert nur 2 Minuten und ist kostenlos!

Code bewerten (Editor)

N

NetzSchleicher

Mitglied
Moin Moin

Nach einer etwas längeren Pause vom Programmieren habe ich mich aus Langeweile mal wieder rangesetzt und noch bin ich mir nicht schlüssig, was genau ich mit der Seite anfangen möchte.

Trotzdem möchte ich vorab schonmal ein Urteil über den Code haben, da dieser eventuell jetzt schon überholt werden könnte.

http://mizaki.mi.funpic.de

Wie gesagt, bitte nur den Code bewerten, Farbgebung und Ausrichtung werde ich sowieso noch einmal überdenken. :D

Grüße, NetzSchleicher
 
Werbung:
Kein Anspruch auf Vollständigkeit:

  • Die Seite ist nicht valide. (Den Validator hast du ja selbst verlinkt, deshalb kein weiterer Kommentar dazu.)
  • Die Header "Content-Style-Type" und "Content-Script-Type" hatte ich noch nie gesehen (es gibt sie, ich weiß). Aber ich gebe dir mal das "benefit of the doubt", dass das browserübergreifend funktioniert und irgendeinen Vorteil hat.
  • Die "Toolbar" könntest du als Liste (<ul>) auszeichnen, etwa mit links-gefloateten <li>-Elementen.
  • Eine Tabelle zur Formatierung der Formularfelder ist nicht unbedingt korrekt. Da gibt es CSS-Alternativen (Suchbegriff etwa: "CSS styling forms").
  • Die Darstellung betreffende Inline-Styles und -Attribute (z. B. <td width="50%">, <div class="textwrapper" style="font-size:10px; line-height:12px; text-align:center;">
    ) sollten nach Möglichkeit in ein Stylesheet ausgelagert werden.
  • <br /> ist nicht dazu gedacht, vertikalen Abstand zu erzeugen. Nutze Die CSS-Eigenschaften padding bzw. margin.
 
mermshaus schrieb:
Die Seite ist nicht valide. (Den Validator hast du ja selbst verlinkt, deshalb kein weiterer Kommentar dazu.)
Zum Vergrößern anklicken....
Jo den hatte ich noch verlinkt als die Liste nicht drin war. Habe das behoben soweit es ging. Gegen das Script von Funpic kann ich aber leider nichts machen.
mermshaus schrieb:
Die Header "Content-Style-Type" und "Content-Script-Type" hatte ich noch nie gesehen (es gibt sie, ich weiß). Aber ich gebe dir mal das "benefit of the doubt", dass das browserübergreifend funktioniert und irgendeinen Vorteil hat.
Zum Vergrößern anklicken....
Ich habe sie bis jetzt immer eingebaut, weil meines Wissens nach das sogar Pflicht nach dem Validator war. Man kann es ja dann als "nice to have" ansehen.
mermshaus schrieb:
Die "Toolbar" könntest du als Liste (<ul>) auszeichnen, etwa mit links-gefloateten <li>-Elementen.
Zum Vergrößern anklicken....
Werde ich mich noch einmal dranbegeben, wobei das für mich eigentlich keinen großen Unterschied macht. :D
mermshaus schrieb:
Eine Tabelle zur Formatierung der Formularfelder ist nicht unbedingt korrekt. Da gibt es CSS-Alternativen (Suchbegriff etwa: "CSS styling forms").
Zum Vergrößern anklicken....
Werde mich direkt mal auf die Suche begeben, neues Wissen ist immer gut.
mermshaus schrieb:
Die Darstellung betreffende Inline-Styles und -Attribute (z. B. <td width="50%">, <div class="textwrapper" style="font-size:10px; line-height:12px; text-align:center;">
) sollten nach Möglichkeit in ein Stylesheet ausgelagert werden.
Zum Vergrößern anklicken....
Gut, muss ich mir mal angewöhnen auch so Kleinigkeiten auszulagern. Danke für den Hinweis.
mermshaus schrieb:
<br /> ist nicht dazu gedacht, vertikalen Abstand zu erzeugen. Nutze Die CSS-Eigenschaften padding bzw. margin.
[/list]
Zum Vergrößern anklicken....
Da ist mir nicht ganz klar, wie ich das anstellen sollte. Etwa eine Klasse mit margin-top o.Ä. einführen und damit die Elemente versehen?

//EDIT:
Habe mal soweit alles geändert, was mir so aufgefallen ist und habe mich an die Ratschläge von mermshaus gehalten. Sonst noch Verbesserungsvorschläge oder fällt noch etwas auf? :D

Am Rande: Ich habe mich auch mal über die Content-Style-Type und Content-Script-Type Debatte erkundigt und es ist wirklich nur ein "nice to have". W3C empfiehlt es zwar, aber jeder browser hat die von mir eingetragenen Values eh als Standard, sodass man sie auch weglassen könnte.
 
Zuletzt bearbeitet:
Werbung:
Ich würde es im Footer zum Beispiel ungefähr so machen:

Code: 
<div class="footer textwrapper">
    <div class="status"><strong>Site Status:</strong> Wartungsarbeiten</div>

    <div class="validation">
        <a href="http://jigsaw.w3.org/css-validator/check/referer">
            <img src="http://jigsaw.w3.org/css-validator/images/vcss-blue"
                 alt="CSS ist valide!" />
        </a>

        <a href="http://validator.w3.org/check?uri=referer">
            <img src="http://www.w3.org/Icons/valid-xhtml10-blue.png"
                 alt="xHTML ist valide!" />
        </a>
    </div>

    <div class="version">Version: 0.2</div>
</div>

Code: 
.footer .status { margin-bottom: 30px; }
.footer .validation { margin-bottom: 30px; }
 
mermshaus schrieb:
Ich würde es im Footer zum Beispiel ungefähr so machen:

Code: 
<div class="footer textwrapper">
    <div class="status"><strong>Site Status:</strong> Wartungsarbeiten</div>

    <div class="validation">
        <a rel="nofollow" href="http://jigsaw.w3.org/css-validator/check/referer">
            <img src="http://jigsaw.w3.org/css-validator/images/vcss-blue"
                 alt="CSS ist valide!" />
        </a>

        <a rel="nofollow" href="http://validator.w3.org/check?uri=referer">
            <img src="http://www.w3.org/Icons/valid-xhtml10-blue.png"
                 alt="xHTML ist valide!" />
        </a>
    </div>

    <div class="version">Version: 0.2</div>
</div>

Code: 
.footer .status { margin-bottom: 30px; }
.footer .validation { margin-bottom: 30px; }
Zum Vergrößern anklicken....

Ist es denn sinnvoller mehrere Klassen für unterschiedliche Divs zu benutzen anstatt nur einen Absatz einzubauen?
Für mich wäre das überflüssiger Code :)
 
Über sowas lässt sich in den Details endlos diskutieren und es ist auch irgendwie Geschmackssache bzw. von weiteren Faktoren abhängig, die nur der Autor selbst im Kopf hat (zum Beispiel, ob die entsprechenden Bereiche später vielleicht noch um anderen Inhalt erweitert werden sollen oder ob sie mehrere Absätze enthalten können oder ob sie ein individuelles Aussehen bekommen sollen oder ähnliches).

<br> ist aber meiner Ansicht nach definitiv inkorrekt an der Stelle.

- 4.6 Text-level semantics &mdash; HTML5 (including next generation additions still in development)
 
Werbung:
mermshaus schrieb:
Über sowas lässt sich in den Details endlos diskutieren und es ist auch irgendwie Geschmackssache bzw. von weiteren Faktoren abhängig, die nur der Autor selbst im Kopf hat.

<br> ist aber meiner Ansicht nach definitiv inkorrekt an der Stelle.
Zum Vergrößern anklicken....
Nachdem ich das gelesen habe ist mir klar, dass <br> wirklich nicht richtig ist, aber ich denke ich bleibe lieber bei Absätzen. :)
 
mermshaus schrieb:
Ich würde es im Footer zum Beispiel ungefähr so machen:
....n bisls code....
Zum Vergrößern anklicken....

Ich würde das eher so machen:..

Code: 
<div [B]id[/B]="footer">
    <h[I]x[/I]>Site Status:</h[I]x[/I]> <p>Wartungsarbeiten</p>

    [B]<ul>[/B]
       [B]<li>[/B]
        <a href="http://jigsaw.w3.org/css-validator/check/referer">
            <img src="http://jigsaw.w3.org/css-validator/images/vcss-blue"
               [B]width="xxx" height="xxx"[/B]  alt="CSS ist valide!" />
        </a>
      [B] </li>
       <li>[/B]
        <a href="http://validator.w3.org/check?uri=referer">
            <img src="http://www.w3.org/Icons/valid-xhtml10-blue.png"
                 [B]width="xxx" height="xxx"[/B] alt="xHTML ist valide!" />
        </a>
[B]       </li>
    </ul>[/B]

    <[B]p[/B] class="version">Version: 0.2<[B]/p[/B]>
</div>

Ansonsten..kann die seite nicht betrachten..würde gesperrt der Zugriff ;)


Grüßli...
Loon3y
 
Werbung:
Also ich hab auch ein Wartungsarbeiten.
HTML: 
DB: Now saving IP, Referer and Timestamp of  every session
Hast du das selbst geschrieben? Ich freue mich ja immer, wenn ich im Internet ein Text lese, in dem auf Groß-/Kleinschreibung geachtet wird. Aber im Englischen schreibt man Nomen klein. Referer und timestamp sind keine Eigenworte, somit werden die kleingeschrieben.
 
Werbung:
Körnerbrötchen schrieb:
Also ich hab auch ein Wartungsarbeiten.
HTML: 
DB: Now saving IP, Referer and Timestamp of  every session
Hast du das selbst geschrieben? Ich freue mich ja immer, wenn ich im Internet ein Text lese, in dem auf Groß-/Kleinschreibung geachtet wird. Aber im Englischen schreibt man Nomen klein. Referer und timestamp sind keine Eigenworte, somit werden die kleingeschrieben.
Zum Vergrößern anklicken....

Jojo ich bin leider ein Großschreibefanatiker und mache es eigentlich an jedem Ort, an dem ich es irgendwie tun kann :p
 
Du musst dich einloggen oder registrieren, um hier zu antworten.

Neueste Beiträge

Zurück
Oben