Du bist nicht angemeldet.

1

Dienstag, 30. Dezember 2003, 08:51

tuxgames.de

Hi,

ich hab vor ein paar Tagen die Seite http://www.tuxgames.de eröffnet. Was haltet ihr von Design?

MfG
RS

2

Dienstag, 30. Dezember 2003, 12:23

Hallo Tuxgames!
Auch gerade erst angemeldet? ;)
Da ich leider nicht allzuviel Zeit habe und deshalb keine komplette "Manöverkritik" abgeben kann, erwähne ich hier nur zwei Punkte, die mir direkt beim ersten Draufschauen auf deine Seite aufgefallen sind:
  1. Du benutzt Frames, was heute eigentlich nicht mehr zeitgemäß ist, siehe dazu z. B: http://www.memowe.de/webauthoring/frames/
  2. Ich vermisse auf allen deinen Seiten eine Doctype-Angabe, eine der wichtigsten Sachen an HTML-Seiten.
    [/list=1]
    Nimm es nicht zu schwer mit meiner Kritik, eigentlich sollte ich hier nicht so anfangen (ist mein erster Beitrag!). Vielleicht finde ich demnächst Zeit für eine ausführlichere Kritik.

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »MyWege« (30. Dezember 2003, 12:54)


3

Dienstag, 30. Dezember 2003, 13:52

Hi tuxgames,
willkommen an Board!
Und MyWege auch ein herzliches Willkommen an Board.

Ich muss leider sagen, das ich die Website vom Design her nicht schön finde. Die Kritikpunkte die mir aufgefallen sind:
Testsystem: MS Windows 98, Internet Explorer 6.0.2800.1106 (Korintenka**er :P ), kein JavaScript

  • Das Logo oben rechts sieht nicht schön aus, die Kante rechts ist hart und könnte Antialiasing vertragen.
    Die Schrift im Logo ist unscharf und das tuxgames und das .de sind zu weit auseinander gezogen.
    Und was bedeutet das "g4l"?
  • Der Banner rechts vom Logo hängt förmlich in der Luft, ohne Zusammenhang. Man könnte diesen schön in einen Header einbauen. So wirkt der Banner, als wäre er eben schnell noch in die Website gequetscht um an einer Banner-Rotation teilzunehmen oder den Webspace zu verblilligen, etc.
  • Wie bereits kritisiert wurde, verwendest du Frames. Die sind wirklich nicht mehr Zeitgemäß. Wie ich zudem gesehen habe nutzt du PHP um deine Inhalte zu verwalten (wenn ich das richtig gesehen habe). Dann noch Frames zu nutzen ist schwachsinn. PHP braucht keine Frames mehr und kann das Menü auch ohne Frame sehr gut und effektiv verwalten.
  • In deinem Menü gibt es zwei mal Links auf deine Administrations-Fläche. Was haben deine User in einem geschützen Bereich von dir zu suchen? Warum Links du Seiten, die für die User eigentlich unsichtbar sein sollten? Ergo: Raus mit den Links!
  • Dein HTML ist wirr und sehr fehlerhaft. Überprüfe deine Website mal im Valdiator von W3C, dort werden alle deine Fehler aufgezeigt, mit dazu gehörigen erklärungen um diese zu beheben: http://validator.w3.org/
  • Du benutzt eine Serifen Schriftart, die sehr schlecht zu lesen ist, besser wäre eine Sans-Serifen Schriftart, ala Arial, Verdana und Helvetica.
    In CSS würde das so außen: font-family: Arial, Verdana, Helvetica, sans-serif;
    Diesen Schnippsel einfach in der Body-Definition in deinem Stylesheet einfügen und deine Texte sind viel lesbarer.
  • Ich würde dir sowie so raten, deine gesamten Design-Angaben in eine externe CSS-Datei zu schreiben, das macht den Quelltext schlanker, deine Website kann schneller geladen werden und man spart sich jede menge arbeit, weil man die Angaben aus der CSS-Datei in jeder Seite verwenden kann, ohne sie immer neu schreiben zu müssen.
    Damit du das am effizientesten ausnutzen kannst, würde ich dir raten, logisches und struckturiertes HTML zu schreiben. Dann brauchst du auch keine komplexen Tabellen Designs mehr, sondern nur noch gut struckturiertes HTML und CSS.
    Alles was dafür notwendig sein sollte, damit du das gut schaffst, findest du in SelfHTML. Und damit du siehst was man mit CSS alles anstellen kann, empfehle ich dir noch einen besuch bei CSS Zen Garden.

So weit von mir, ich hoffe du kannst was mit meiner Kritik anfangen und ich konnte dir was helfen. Bei nachfragen stehe ich gerne zu verfügung :) .

MfG Arne "Nash" Kamola :)

4

Dienstag, 30. Dezember 2003, 16:58

Hi,
zu dem bereits Gesagtem ist eigentlich nichts mehr hinzuzufügen. Eine Anmerkung aber trotzdem zur Schrift:
Hier wird _keine_ Schriftart definiert. Also kann man auch nicht pauschal sagen, daß die Schrift schlecht lesbar wäre. Vielmehr wird (sicherlich jedoch in Unkenntnis der Möglichkeiten) dem Besucher abverlangt, in seinem Browser eine _lesbare_ Schriftart als Default einzustellen.
Die entsprechende Kritik bestätigt jedoch, daß dies keine gute Idee ist..;-)
Gruß
Ingo

5

Dienstag, 30. Dezember 2003, 23:19

hi,

danke schon mal für die ganze konstruktive kritik.

das mit den frames stimmt eigentlich. ich werd mich demnächst mal dransetzten und versuchen, das etwas umzubauen. wird allerdings denke ich ein ganzes stückchen arbeit da doch relativ viele dateien angepasst werden müssen.

wenn ich dann schon dabei bin werd ich das mit den doctype-angaben wohl auch noch nachholen.

das g4l bedeuetet games for linux und ist ein überbleibsel einer alten domain. dass das tuxgames und das de so weit auseinandergezogen sind, liegt unter anderem daran, dass sich die beiden grafiken in zwei unterschiedlichen frames befinden und dass man mehr oder weniger erkennen soll, dass sowohl tuxgames und g4l eine .de-Adresse sind.

hmm, mit dem banner fällt mir momentan keine wirkliche lösung ein, da ich keine ahnung habe wie ich ihn in das design integrieren könnte.

die links zu den admin-menü's gibt es deshalb weil die user diese durchaus benutzen sollen. die seite ist so aufgebaut, dass die user ihre eigenen links posten können. ich hab das ganze jetzt mal zu user-login umgenannt. ich denke dadurch sollte es etwas übersichtlich sein.
den zweiten admin link, der wirklich nur für administratoren bestimmt ist, werd ich später evtl entfernen oder an einer anderen stelle platzieren.

danke für die tipps, ich werde versuchen sie in den nächsten tagen umzusetzen und mich dann nochmal melden.

mfg
rs

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »tuxgames« (30. Dezember 2003, 23:30)


6

Mittwoch, 31. Dezember 2003, 11:16

Hi,

ich hab ein Problem, ich hab die Seite jetzt teilweise umgebaut, so dass kein Frames mehr enthalten sind. Hier kann man das ganze vorab schonmal betrachten: http://217.84.49.176/cms2/cms/index2.php (geht nur 24 Stunden da auf meinem Home-Webserver.)

Jetzt versuche ich noch die Seite nach möglichkeit dem HTML-Standart anzupassen.
Doch bei einigen Sachen komme ich einfach nicht weiter:
http://validator.w3.org/check?uri=http%3…ms%2Findex2.php

Feher in Z 12: Was soll das bedeuten? Ich hab keine Ahnung !!!


Fehler Z 17: Damit sich die Navigation nicht immer mit nach unten verschiebt wenn der Inhalt der Seite größer wird, habe ich <tr valign='top'> eingefügt um die Tabelle oben zu halten. Nur leider scheint dies nicht wirklich erlaubt zu sein. Gibt's da eine andere Möglichkeit.


Genauso in Z.22: mit dem <td background=''>. Wie kann ich anders eine Hintergrundgrafik in eine Tabelle einfügen.

Wie auch mit <td heigth=''> . Laut http://selfhtml.teamone.de/html/tabellen…#breiten_hoehen funktioniert es so. validator.w3.org sagt jedoch das es falsch ist. Was stimmt den nun ?!? ?( ?( ?(

Danke
RS

7

Mittwoch, 31. Dezember 2003, 12:53

hi,
zunächst: du kannst (und solltest) tabellen über CSS formatieren.

dann: sieh die fehler im kontext. der erste fehler z.b. in zeile 11 erklärt sich über die zeile 9 darüber:

9: <font face="arial, courir">
10:
11: <table border=0 bgcolor='#0E3259' cellpadding='0' cellspacing='0' width='100%'>

abgesehen davon, daß du font sowieso nicht mehr verwenden solltest, ist dies ein inline-element, in welchem nur andere inline-elemente vorkommen dürfen.
gruß
Ingo

8

Mittwoch, 31. Dezember 2003, 13:22

Zitat

Original von tuxgames
Hi,

ich hab ein Problem, ich hab die Seite jetzt teilweise umgebaut, so dass kein Frames mehr enthalten sind. Hier kann man das ganze vorab schonmal betrachten: http://217.84.49.176/cms2/cms/index2.php (geht nur 24 Stunden da auf meinem Home-Webserver.)

Jetzt versuche ich noch die Seite nach möglichkeit dem HTML-Standart anzupassen.
Doch bei einigen Sachen komme ich einfach nicht weiter:
http://validator.w3.org/check?uri=http%3…ms%2Findex2.php

Feher in Z 12: Was soll das bedeuten? Ich hab keine Ahnung !!!

Hat Ingo schon gesagt, table ist ein ein wasweißichwas für ein Element, das nicht in inline-Elementen wie font enthalten sein darf.


Zitat

Fehler Z 17: Damit sich die Navigation nicht immer mit nach unten verschiebt wenn der Inhalt der Seite größer wird, habe ich <tr valign='top'> eingefügt um die Tabelle oben zu halten. Nur leider scheint dies nicht wirklich erlaubt zu sein. Gibt's da eine andere Möglichkeit.
CSS, style="vertical-align: top;"


Zitat

Genauso in Z.22: mit dem <td background=''>. Wie kann ich anders eine Hintergrundgrafik in eine Tabelle einfügen.

Per CSS, z.B. so: <table style="background: url(image.jpg);" ... >

Zitat

Wie auch mit <td heigth=''> . Laut http://selfhtml.teamone.de/html/tabellen…#breiten_hoehen funktioniert es so. validator.w3.org sagt jedoch das es falsch ist. Was stimmt den nun ?!?
Beides stimmt. Bei Selfhtml steht, dass es dieses Atrribut bei HTML3 und XHTML1.0(wahrscheinlich nur transitional) gibt, du verwendest aber HTML4 , da wurde dieses Attribut abgeschafft. Stattdessen kannst du CSS verwenden style="height: 30px;"
Signatur von »Darii« Against TCPA - TCPA heißt Freiheit verlieren
D.A.R.I.I.: Digital Artificial Replicant Intended for Infiltration

Dieser Beitrag wurde bereits 2 mal editiert, zuletzt von »Darii« (31. Dezember 2003, 13:23)


9

Mittwoch, 31. Dezember 2003, 18:31

hi,
zur Ergänzung:

table ist ein ein wasweißichwas für ein 'blocklevel' Element ;-)

das Attribut 'height' ist - für TD - sowohl in HTML 4.01 als auch XHTML 1.0 zugelassen; bei beiden Standards jedoch nur in den transitional-Varianten.
ein Attribut 'heigth' gibt es jedoch tatsächlich nicht ;-)

Gruß
Ingo

10

Donnerstag, 1. Januar 2004, 15:29

Hi,

ich hab bei der Seite jetzt die Frames entfernt und auch die Fehler weitestgehen ausgebessert. Die anderen Tipps werde ich in demnächst auch noch befolgen, jetzt mach ich aber erst mal für ein paar Tage Urlaub.

Achja einen Fehler hab ich noch:
Ich han folgenden Code eingebunden (stammt von count24.de):

<script language='javascript'>

Er bemängelt, dass type fehlt. Was soll ich als type angeben?

Danke für die Tipps
RS

11

Donnerstag, 1. Januar 2004, 16:18

Das muss angegeben werden:

Quellcode

1
<script type="text/javascript">

language="javascript" wird zwar noch oft verwendet, ist aber nicht mehr richtig, oben genanntes ist richtig. language="javascript" ist ausgemusstert worden.

MfG Nash :)

12

Donnerstag, 1. Januar 2004, 18:17

ok, danke.
jetzt sollten keine fehler mehr im code sein.

Dieser Beitrag wurde bereits 1 mal editiert, zuletzt von »tuxgames« (1. Januar 2004, 18:37)