Feature Request: Einführen von Templates

[h8h]

Hi,

mittlerweile wird popupcontent.js immer größer. Das ist generell kein Problem, nur leidet meiner Ansicht die Übersichtlichkeit und die Möglichkeit designmäßig Anpassungen zu machen immer mehr darunter.

Deshalb der "Feature request" Code und View (Template) voneinander zu trennen.

Folgendes habe ich jetzt geändert:

• Verwendung von jsrender Bibliothek
• Implementierung des popcontent als Template in die index.html, siehe script type="text/x-jsrender"
• Auslagern der Übersetzung, der Adressdarstellung in die update_data.js, damit ändert sich nun auch das Aussehen der details.json, die sieht jetzt so aus:

   "id" : "node/1329846483",
   "type" : "Feature",
   "name" : "Hofladen Nitzinger Eggstätt Bioladen",
   "geometry" : {
      "type" : "Point",
      "coordinates" : [
         12.3855076,
         47.9198983
      ]
   },
   "property" : "farm",
   "properties" : {
      "Telefon" : "+49 8056 909644",
      "Biologisch" : "nur",
      "shop" : "farm"
   },
   "address" : {
      "postcode" : "83125",
      "city" : "Eggstätt",
      "housenumber" : "25",
      "street" : "Chiemseestraße"
   },
   "opening_hours" : "Mo-Sa 08:30-12:00; Mo-Tu 14:00-18:00; Th-Fr 14:00-18:00"
}

Die Idee dahinter, properties nur noch auf die uns "unbekannten" Variablen begrenzen. properties wird später im Template, wie zuvor auch, in "Weitere Daten gerendert".

• Implementierung von Öffnungszeiten. Habe erst heute deine Commits zu den Öffnungszeiten gesehen, deshalb habe ich das auch noch kurz hinzugefügt. Da die Auswertung der Öffnungszeiten dynamisch erfolgen muss, wird dem details.json JSON dynamisch noch das Feld open hinzugefügt, das geschieht dann in der js/popupcontent.js.
• Abschließend habe ich dann nochmal ein Datenabgleich laufen lassen und vermtl. auch den Fehler mit dem "cannot open ..." behoben.

Hoffe du bist nicht allzusehr schockiert :)

Anschauen könntest du es wieder hier acme@direktvermarkter

Grüße
Chris

[stefangrotz]

Hey tolle Arbeit :) jsrender muss ich mir unbedingt anschauen. Evtl baue ich damit auch die Inhalte im Menü damit man die auch mehrsprachig machen kann.

Aus irgendeinen Grund funktioniert das Try-Catch für die Öffnungszeiten nicht immer, hier z.B. nicht:

https://acme.im/direktvermarkter/#49.00865,8.37582,18z

https://farmshops.eu/#49.00896,8.37555,18z

Bei diesem Popup gibt es einen Fehler weil die Bibliothek PH nicht kennt.

Ich denke man kann die Funktion mit den öffnungszeiten auch einfach mit in die index html hinein packen, dafür braucht man keine eigene popupconten.js

[h8h]

Danke :).

Bzlg. Öffnungszeiten, da hab ich grad mal nen Issue aufgemacht.

jsrender, bietet nur einfache Methoden, wie if/else, for, keine try catchs.

Es würde somit aus meiner Sicht mehr Sinn machen, das das Auswerten der Öffnungszeiten weiterhin irgendwo im Code stattfindet und das letztendliche anzeigen dann im Template.

Man könnte aber die js/popupcontent.js durchaus vereinfachen, in dem man die oh.getState() direkt ins feature array reinschreibt.

Ich würde noch gerne das Template auslagern in eine Datei, doch das funktoniert irgendwie nicht so einfach.

Mit mehrsprachigen Menüs klingt super. Dann würde ich evtl. den Text in ein JSON Blob auslagern und dann so etwas in bauen:

 <ul>
    {{props menu}}
                    <li>
                     <a href={{:prop}}>{{:key}} </a>
                    </li>
     {{/props}}
</ul>

menu_de.json

   "menu" : {
      "Startseite" : "index.html",
      "Über" : "about.html",
   }

[stefangrotz]

Gibt es irgendwelchen besonderen gründe warum Du dich ausgerechnet für jsrender als Template Engine entschieden hast? Ich habe grundsätzlich nichts dagegen, es ist nur eine der kleineren Lösungen deswegen interessiert mich das.

[h8h]

Wegen der kleineren Codebasis habe ich die genommen. Für die paar Templates, dachte ich brauchen wir keine Monsterengine.

Ich habe mich etwas informiert und hab einen Ansatz gefunden, sowas recht schlank umzusetzen. Der Beitrag war aber von 2008 oder so. Und aus diesem Ansatz ist dann jsviews entstanden.

Und jsviews ist echt nicht viel Code.

Wenn du aber mehr brauchst oder größere Pläne hast, können wir auch gerne Umswitchen.

Mir ist nur das Prinzip wichtig, das wir Code und Desgin auseinander halten.

[stefangrotz]

Klingt logisch. Ich habe auf lange Sicht ein anderes Design im Hinterkopf aber das müsste auch mit der Bibliothek gehen. Ich gehe da gerne mit und experimentiere in den nächsten Tagen ein wenig damit herum.

[h8h]

Perfekt. Danke.
Gibts da schon konkrete Vorstellungen :)

Ich würde ja gerne noch eine Detailansicht haben.

Also eine Ganzseitige, ähnlich wie bei Wheelmap.

Auf einer solchen, könnte man dann perfekt auch Formular bauen, zur Änderung der Daten.