search

ribajs / riba

Lightweight and powerful data binding + web components + templating + routing solution
https://ribajs.com
MIT License
14 stars 2 forks source link

Leaflet #6

Closed dnil-io closed 4 years ago

dnil-io commented 4 years ago

Added a leaflet component

JumpLink commented 4 years ago

Innerhalb von /demos/rv-leaflet eine yarn.lock anlegen (touch yarn.lock) damit darin yarn install funktioniert, ansonsten meckert yarn, dass das Projekt nicht zum Workspace gehört.

JumpLink commented 4 years ago

Da die Komponente leaflet-map heißt, die Demo am besten genauso benennen (statt rv-leaflet).

JumpLink commented 4 years ago

Vllt wäre es ganz schön die Möglichkeit zu haben im <marker ..> angeben ob es Tooltip / Popup geben soll (Lösungsvorschlag wenn es kein Text gibt, dann gibt es auch kein Tooltip). Es sieht im Code so aus als könnte man auch eigene Icons angeben? Wäre schön das in der Demo mit einem zweiten Marker zu demonstrieren. Z.B unser Pixelherz beim Standort unseres Büros ;)

dnil-io commented 4 years ago

Vllt wäre es ganz schön die Möglichkeit zu haben im <marker ..> angeben ob es Tooltip / Popup geben soll (Lösungsvorschlag wenn es kein Text gibt, dann gibt es auch kein Tooltip). Es sieht im Code so aus als könnte man auch eigene Icons angeben? Wäre schön das in der Demo mit einem zweiten Marker zu demonstrieren. Z.B unser Pixelherz beim Standort unseres Büros ;)

Das habe ich sogar schon gemacht :) Einfach ein openByDefault="true/false", aber habs jetzt mal mit ins example getan. Das mit dem Icon kann ich auch ganz schnell machen :+1:, da gibt es aber mehrere möglichkeiten: Entweder man macht einmal im template ein element, das dann wiederverwendet werden kann, oder man gibt die eigenschaften vom icon jedes mal im marker an. letzteres wäre aber finde ich eher unübersichtlich, da es ganz schön viele eigenschaften sind: "markerIcon, markerShadow, markerIconSize, markerShadowSize, markerIconOffsetX+Y, markerShadowOffsetX+Y. Das erstere ist natürlich nicht ganz so schön wenn es im dom ist, da es ja kein richtiges element ist, man könnte das auch als json array an die map übergeben. Glaube aber das ich den Weg nehme, da es am übersichtlichsten ist. Kanns aber auch noch ändern wenn es dir dann doch nicht gefällt.