search

numero2 / contao-storelocator

Contao Plugin for managing stores (or in common address data) and providing a frontend-search based on geo data.
https://www.numero2.de/contao/erweiterungen/storelocator.html
GNU Lesser General Public License v3.0
13 stars 8 forks source link

Styleguide Templates #12

Closed andreasisaak closed 11 years ago

andreasisaak commented 11 years ago

Ich weiß dass das auch sehr vom Geschmack des Entwicklers abhängt aber in der Regel hat sich bei Contao durchgesetzt das man grade in Templates die alternative Schreibweise nutzt:

http://php.net/manual/de/control-structures.alternative-syntax.php

Dadurch werden die Templates lesbarer und man weicht nicht so sehr vom Core ab.

bennyborn commented 11 years ago

Werde ich gerne beim zukünftigen Updates berücksichtigen. Der Storelocator war eine unserer ersten Contao-Erweiterungen, da waren wir noch nicht ganz mit den Styles vertraut :)

andreasisaak commented 11 years ago

Hab ich mir schon gedacht. Deswegen bemängele ich auch noch nicht die Coreklassen der Extension. Da gäbe es auch noch etliches aufzuräumen ;) Aber nimm meine Tickets bitte als Hilfe wahr, nicht als Kritik.

bennyborn commented 11 years ago

Hehe nene Kritik und Hilfen sind immer gut. Jedoch würde mich jetzt doch schon noch interessieren was Du gerne an den Klassen anders machen würdest ;)

andreasisaak commented 11 years ago

Ich hangel mich grade durch alle Bereiche. Musst dich noch etwas gedulden :)

bennyborn commented 11 years ago

Aufgeräumt in 62481021d4e79583039fc5240b85d151af285d14