Skip to content

Conversation

SubiNesa
Copy link
Contributor

No description provided.

…sing `L.AwesomeMarkers.icon` on startIcon and endIcon
@SubiNesa
Copy link
Contributor Author

#161

@mpetazzoni
Copy link
Owner

Thanks for the PR! This breaks setting icons for categories/waypoint types though, which are expected to be set with:

markers: {
    wptIcons: {
      'Coffee shop': new L.AwesomeMarkers.icon({
        icon: 'coffee',
        prefix: 'fa',
        markerColor: 'blue',
        iconColor: 'white'
      })
    }
}

Can you find a way to tell those situations apart in _prepare_markers() ?

@SubiNesa
Copy link
Contributor Author

SubiNesa commented Sep 13, 2024

Thanks for your comment.
I am little bit confused, wouldn't that be this line https://github.com/mpetazzoni/leaflet-gpx/blob/main/gpx.js#L292 ?

I could add new condition for startIcon and endIcon, but not sure if that would be overkill

@mpetazzoni
Copy link
Owner

Ah yes you're right, I already had to handle that case because type(null) === 'object'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants