Valides Xhtml
 

Hi!
Ich nutze das CalitrixWiki und bin echt begeistert! :D
Leider spuckt der Validator vom W3C (http://validator.w3.org/) noch einige errors aus. :eek:
Ich hab mir das mal angeguckt und festgestellt dass es im Grunde zwei Probleme sind:

1. <h1>, <ul> und <li> werden innerhalb von <p></p> benutzt.
das liegt an zeile 121 in der class_parser.php.

2. Sonderzeichen werden in URLs nicht encodiert. Z.b. das "&" in
"./cwiki.php?page=Wiki:Users&user=1". dieses müsste eigentlich als "&amp;" im Quellcode stehen.

Das zweite Probem ist nicht dramatisch und weit verbreitet, das erste stört mich schon mehr.
Wenn man in zeile 121 die <p>s durch <br />s ersetzt, geht es zwar, aber das layout (und struktur der seiten) leidet etwas.
Zeile 121:
Und mit <br />s:
Wär schön, wenn man das irgendwie in den Griff bekommen könnte.
Es wird zwar in allen meinen Browsern richtig angezeigt, aber ein sauberer Code ist gerade bei einem Wiki sinnvoll.

Zum ersten Problem: Überschriften und Listen sind da nicht die einzigen. Auch Tabellen dürften in dem Zusammenhang Probleme machen. Man könnte sicherlich bei solchen Elementen prüfen ob sie innerhalb eines Absatzes stehen und gegebenenfalls den Absatz schliessen, das würde allerdings einigen Mehraufwand beim Parsen bedeuten. Ich muss mal schauen wie und ob sich das effizient machen lässt.

Zum zweiten Problem: das ist ein kleiner Bug in der Funktion die die URL generiert. Normalerweise werden & durch &amp; dargestellt, nur das erste & nicht ;) In der nächsten Version ists gefixt.

libs/class_core.php, in Zeile 502

ersetzen durch

... sollte so funktionieren, falls du es selber schonmal fixen willst ;)

wow. das nenn ich mal schnellen support :D
danke!

Zu dem Problem beim Ersetzen von <p> durch <br />: ersetz mal probeweise die <p>s durch Grund: ein <br /> beispielsweise am Ende eines Absatzes wird meist auch nicht dargestellt, weil eine Zeile ohne Zeichen halt weiter keine Höhe hat. Wohingegen auch ein nichtsichtbares Leerzeichen eine Zeichenhöhe besitzt. ;)

Man könnte, anstatt jeden Absatz in <p>...</p> zu setzen auch einfach zwei <br /> bei jeder leeren Zeile einsetzen. Das Ergebnis im Browser dürfte das gleiche sein, nur wäre es semantisch kein gutes XHTML mehr.

dann bleibt wohl keine andere Alternative als den Parser dazu zu bringen vor einer Überschrift, Liste, Tabelle etc. den Absazu zu schließen und falls danach noch Text folgt (bei Überschriften wohl fast immer, bei Tabellen ja nicht unbedingt) einen neuen zu öffnen...