Code bewerten (Editor)

Community Showcase & Websitecheck
1

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. :smiley:

Grüße, NetzSchleicher

2

Kein Anspruch auf Vollständigkeit:

[ul]
[li] Die Seite ist nicht valide. (Den Validator hast du ja selbst verlinkt, deshalb kein weiterer Kommentar dazu.)
[/li][li] 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.
[/li][li] Die „Toolbar“ könntest du als Liste (

    ) auszeichnen, etwa mit links-gefloateten
  • -Elementen.
    [/li][li] Eine Tabelle zur Formatierung der Formularfelder ist nicht unbedingt korrekt. Da gibt es CSS-Alternativen (Suchbegriff etwa: „CSS styling forms“).
    [/li][li] Die Darstellung betreffende Inline-Styles und -Attribute (z. B. ,

    [/li]) sollten nach Möglichkeit in ein Stylesheet ausgelagert werden.
    [li]
    ist nicht dazu gedacht, vertikalen Abstand zu erzeugen. Nutze Die CSS-Eigenschaften padding bzw. margin.
    [/li][/ul]

3

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.

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.

Werde ich mich noch einmal dranbegeben, wobei das für mich eigentlich keinen großen Unterschied macht. :smiley:

Werde mich direkt mal auf die Suche begeben, neues Wissen ist immer gut.

Gut, muss ich mir mal angewöhnen auch so Kleinigkeiten auszulagern. Danke für den Hinweis.

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? :smiley:

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.

4

Ich würde es im Footer zum Beispiel ungefähr so machen:

[code]


Site Status: Wartungsarbeiten

<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>
[/code]

.footer .status { margin-bottom: 30px; } .footer .validation { margin-bottom: 30px; }

5

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 :slight_smile:

6

Ü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).


ist aber meiner Ansicht nach definitiv inkorrekt an der Stelle.

7

Nachdem ich das gelesen habe ist mir klar, dass
wirklich nicht richtig ist, aber ich denke ich bleibe lieber bei Absätzen. :slight_smile:

8

Ich würde das eher so machen:…

<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 :wink:

Grüßli…
Loon3y

9

Mh muss ich nich verstehen, bist bis jetzt der Einzige dems gesperrt wurde :open_mouth:

10

login sieht einwenig komishc aus. Änder det mal und ach ja Noob schreibt man ohne k also nciht KNOOB

11

Knoob = Kacknoob, nur am Rande xD

Und ich weis nich, warum der komisch aussieht, genauere Angaben wären hilfreich.

12

Also ich hab auch ein Wartungsarbeiten.
[HTML]DB: Now saving IP, Referer and Timestamp of every session[/HTML]
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.

13

Jojo ich bin leider ein Großschreibefanatiker und mache es eigentlich an jedem Ort, an dem ich es irgendwie tun kann :stuck_out_tongue:

14

Das liegt an Arbeit und div. Proxyeinstellungen… :wink: