[luci] <div /> soup busters step 1

Jo-Philipp Wich xm at subsignal.org
Mi Sep 17 23:13:59 CEST 2008


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Hi Alina.

Danke für den Patch, sieht soweit erstmal gut aus.
Einige Anmerkungen:

 - Den Whitespace zwischen "-%>" und <?xml ...?> musst du nicht löschen, das übernimmt
   bereits der Template-Prozessor

 - Warum <br class="clear" /> durch <div class="clear"></div> ersetzen wenn wir divs
   sparen wollen?

 - Wie stellst du dir eine saubere Implementierung des IE-Helfer-JS vor ohne wesentlich
   mehr JS zu implementieren?


Der Patch sieht ansonsten soweit gut aus, ich kann ihn leider so im Moment nicht anwenden.
Wir können es uns nicht leisten den IE-Support einfach mal vorübergehen rauszuwerfen, die
Nutzer beziehen atm ihre LuCI Snapshots direkt aus den Nightly-Builds, da wäre es ein
unschöner Zug von uns mal eben wissentlich die IE-Kompatibilität zu entfernen. Ohne das
"Hack-Script" würden die Menüs nicht funktionieren was das Interface für IE-Nutzer
faktisch unbrauchbar macht.

Ich könnte dir anbieten das wir ein separates Theme einrichten an dem wir die ganzen
Änderungen austesten um diese dann später zu mergen. Am jetzt funktionierenden Stand würde
ich zumindest am openwrt.org Hauptthema ungern etwas kaputt machen.


Grüße,
JoW
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFI0XMXdputYINPTPMRAjskAJ483AvqGyS5aa/WsFNpPYvaqFtGegCcDrqu
AKnlUfrhF/1iRhtXO+Q69TE=
=U8ri
-----END PGP SIGNATURE-----


More information about the luci mailing list