Update mod_anystores_map.html5

Tastaturberuf / anyStores

Contao-Module: Organize any locations with geolocation search.

GNU Lesser General Public License v3.0

11 stars 7 forks source link

Update mod_anystores_map.html5 #37

Open stefansl opened 9 years ago

stefansl commented 9 years ago

Add routing - man könnte das natürlich auch im Modul konfigurierbar machen. :)

Tastaturberuf commented 9 years ago

Ich kanns nur einbauen wenn es optional ist. Und per default aus.

stefansl commented 9 years ago

Logisch. Wenn du möchtest, kann ich die dca im Modul entsprechend erweitern. Ansonsten lösch den PR einfach.

Tastaturberuf commented 9 years ago

Ich lass den PR hier mal als Erinnerung hängen. Ich hab noch nicht final entschieden ob ich die Eingabe der Adresse mit in der Blase haben will. Eigentlich reicht mir der Link. Und dann sind es so Kleinigkeiten wie z.B. das hl=de hardgecoded ist.

Und wenn es nach mit geht möchte ich den User nicht auf eine andere Seite schicken >.<

Tastaturberuf commented 9 years ago

Die Lösung ist aber gut und funktioniert.

stefansl commented 9 years ago

Du hast Recht. Besser ist es vielleicht eine Erweiterung für AnyStores anzubieten. anystores_routes, oder so.

Tastaturberuf commented 9 years ago

Das auslagern und ist mir am Ende auch zu fehleranfällig. Wer das im Ballon nicht haben will, muss das ebend ausm Template nehmen. Damit muss das eigentlich nichtmal optional sein, denn das ist es damit ja. Ich hätte das rendern der Elemente gerne in einer eigenen Klasse, damit ich das auch direkt im Detail-Modul einbauen kann.