search

FriendsOfREDAXO / geolocation

Tile-Proxy/Cache und Kartendarstellung mit Leaflet für REDAXO 5.13+
MIT License
26 stars 3 forks source link

HTML-Colorcodes für Marker-Farben escapen #69

Closed christophboecker closed 1 year ago

christophboecker commented 1 year ago

Hintergrund: es gibt im JS eine Funktion, die farbige Marker als Inline-SVG erzeugt. Der erzeugte Code ist escaped, also z.B. " ist durch %22 ersetzt. Die an die Funktion übergebenen Farbnamen (z.B. green) werden auch klaglos richtig in den Code eingesetzt. Mit Farbcodes (z.B. `#008800') geht das schief.

Aktueller Workaround: Farbcodes als %23.... (z.B. %23008800) selbst escapen.

Für die nächste Version das JS selbst escapen lassen.

TobiasKrais commented 1 year ago

Wenn du mir sagen kannst wo die Stelle im Code ist, programmiere ich einen PR.

christophboecker commented 1 year ago

Danke für das Angebot. Da ich ohnenhin grade eine großangelegte RexStan-getriggerte Überarbeitung mache, erledige ich das gleich mit. Es gibt jetzt erstmal eh kein Release.

Wenn Du für Dich bis dahin die Sache fixen willst: https://github.com/FriendsOfREDAXO/geolocation/blob/9b37e12b00f8fa4d40ecce5d3fea382da5c80b98/install/geolocation.js#L89-L93 Hier im Code einfach color = encodeURI(color); ergänzen.

Und danach einmal die Einstellungsseite speichern! Dann wird das gesamte CSS/JS neu in das Asset-Verzeichnis compiliert. Sollte jedenfalls ;-)

TobiasKrais commented 1 year ago

Hilft leider nicht. Die Raute bleibt.

TobiasKrais commented 1 year ago

Ich hab den Bugfix gerade getestet. Funktioniert bei mir leider nicht.

christophboecker commented 1 year ago

Danke für den Hinweis. Es musste encodeURIComponent sein und nicht encodeURI. #94

TobiasKrais commented 1 year ago

Getestet. Jetzt funktioniert es korrekt. Danke dir!