Theme f?r Froxlor

[harrydeluxe]

Hallo,

unter https://github.com/h...e/Froxlor-Theme habe ich ein Theme bereit gestellt. Der Gro?teil funktioniert schon, ist aber noch nicht in allen Browsern getestet.

[3bsd]

Hallo,

 

ich habe das Theme angepasst so das keine ?nderungen an den Core-Files n?tig sind und noch ein paar Kleinigkeiten gefixt. Siehe Changelog. Wer noch Interesse hat findet hier das Repository:

https://github.com/cosmomill/Froxlor-Theme

 

Bye

Rene

[emmi]

Hi.

 

Ich hab mir das Theme mal angeschaut. Sieht klasse aus, allerdings bin ich etwas verwirrt.

Warum m?ssen die Dateien in lib angepasst werden? Funktionieren die alten Themes weiterhin? Ist es kompatibel zur 0.9.27-1 bzw. gab es noch ?nderungen zur rc1?

 

Gru? emmi

[blu3bird]

Das funktioniert soweit alles gut. Hab die Datein einfach angepasst (Codestellen rausgesucht). Nur es gibt da noch nen Bug mit dem Tooltip bei der Traffic Anzeige.

Das Tooltip wird irgendwo ganz weit rechts angezeigt.

Aber sonst finde ich, ist es ein super Theme

 

Browser: FireFox 12

[peperoni]

die screenshots sehen wirklich toll aus, allerdings w?re eine Antwort auf die Frage von emmi ganz nett

[d00p]

lib/functions/output/function.buildNavigation.php

 

Hier wurden laut code verschiedene anpassungen gemacht, wenn der theme "Delacap" ist. z.B. bestimmte seiten nur als modalen-dialog darstellen

 

lib/classes/database/class.db.php

 

Hier wurden die variablen-deklarationen statt mit 'var' (php4) in 'private' (php5) ge?ndert. Sollte im Grunde keinen gro?en Unterschied machen. In der query funktion wird auf ein mysql-timeout gepr?ft und falls die verbindung abgebrochen ist, eine neue hergestellt. Das sollte allerdings Aufgabe der Froxlor-Devs sein und nicht eines Themes

 

lib/formfields/customer/email/formfield.emails_edit.php

 

Hier ist eine unsch?ne Anpassung, denn die ?nderung des 'value' wertes im array betrifft damit alle Themes. Vorschlag hier: Die Zeile m?sste komplett in ein eigenes template ausgelagert werden, denn sie enth?lt sowieso HTML-code, sodass jedes Theme sein Layout beibehalten kann.

 

Bsp. Classic und Froxlor-Theme: $lng['panel']['yes'].' [<a href='.$filename.'?page=accounts&action=changepw&id='.$result['id].'&s='.$s.'">'.$lng['menue']['main']['changepassword'].'</a>] [<a href='.$filename.'?page=accounts&action=delete&id='.$result['id].'&s='.$s.'">'.$lng['emails']['account_delete'].'</a>]'

 

Delacap-Theme: '<div class="btn-group"><a class="btn btn-mini" href="'.$filename.'?page=accounts&action=changepw&id='.$result['id'].'&s='.$s.'">'.$lng['menue']['main']['changepassword'].'</a> <a rel="confirm" class="btn btn-mini" href="'.$filename.'?page=accounts&action=delete&id='.$result['id'].'&s='.$s.'">'.$lng['emails']['account_delete'].'</a></div>'

 

lib/classes/output/class.paging.php
lib/formfields/customer/email/formfield.emails_edit.php

 

unerheblich, nur eine Leerzeile am Ende der Datei eingef?gt.

 

Mehr konnte ich so auf die schnelle nicht sehen. Leider sind viele Dateien schon vor dem initial-commit ge?ndert worden, sodass nicht alles nachvollziehbar ist.