Closed jancborchardt closed 8 years ago
Looks nice! :+1: from me A little enhancement: Do we want to have the marker shadow?
I removed it because it was also not visible for the favorite markers. But maybe yes, it’s better to have it to visually separate it from the map. Will adjust.
Any news, @jancborchardt ?
Why does this PR modify a 3rdParty file (i.e. js/3rdparty/leaflet/leaflet-src.js
)?
As soon as I add the shadow back, it won't mess with 3rdparty files anymore I think.
@jancborchardt any update on this?
@jancborchardt ping?
@v1r0x i believe @jancborchardt is on vacation this week
Does he even work? :D
@jancborchardt Any news?
@jancborchardt Hope you are not mad, but I decided to revert the js changes and adjusted the fav icon and added a new shadow marker. @jancborchardt @Henni What do you think? Ready to merge?
@v1r0x nah, I just need to test. :) The favorite marker looks a tad better too.
The SVG files definitely need to be compressed though – using Scour, with the line from core/img/image-optimization.sh, if you want to play around with that. ;)
@jancborchardt I used scout with the settings from the shell script and also moved the files to another folder. Thus all marker icons are in one folder and not in different folders. I'll squash the commits if you (and/or @Henni) are ok with the changes :)
The navigation markers are broken. Otherwise looks good:
Unfortunately the routing machine uses the hardcoded (or taken from leaflet.js
) path to the old .png
file. I tried to copy the current marker as png to the old path, but this distorts the marker icon.
Got it!
Ok, let’s get this in – we can always polish more in future pull requests. :)
Before & after:![capture du 2016-01-03 20-32-40](https://cloud.githubusercontent.com/assets/925062/12080401/30b2e0c6-b259-11e5-973a-4ba7cd529714.png)
More in line with simple ownCloud style. Please review @v1r0x @Henni