nativescript-mapbox
nativescript-mapbox copied to clipboard
Added above/below options to addLayer/addPolygon/addPolyline
I noticed that new layers were being added above the markers. This may not always be desirable. I've added optional params above
and below
.
I've also added above
and below
to addPolygon
and addPolyline
. This resolves #188 & #308.
I've also fixed addSource
, removeSource
, addLayer
, deleteLayer
when calling from an XML map (shoud have tested this initially).
I'm also working on location layer customisations. I'll submit a PR soon.
Looks like a reasonably nice solution to the problems reported 👌 was looking at doing this too
FYI I've been working on bringing this package up to date and resolving the build issues as well, if there were any concerns around merging this despite the build failing. Currently the lint fails due to existing code in the typings (A bunch of rogue readonly
s with nothing after them).
PR opened to resolve the build issues amongst other things. These changes don't break the build (tested locally): https://github.com/EddyVerbruggen/nativescript-mapbox/pull/311
Tested these changes on my branch, all seems 👌
@lochstar my PR was accepted, so you'll need to resolve / rebase probably.
@Codex- Sorted!
@EddyVerbruggen Can we get this merged? :)
Probably won't until the failing build is resolved?
Thanks @Codex-
How do I run the branch containing the addsource and addLayer?
@alexonozor For some reason addSource and addLayer was broken/left out in the last batch of major updates to this branch.
I'm currently working on a fork that has NS7 updates and addSource/addLayer fixes here: https://github.com/codelabau/nativescript-mapbox
@alexonozor For some reason addSource and addLayer was broken/left out in the last batch of major updates to this branch.
I'm currently working on a fork that has NS7 updates and addSource/addLayer fixes here: https://github.com/codelabau/nativescript-mapbox
I referenced the packages like this in my package.json "nativescript-mapbox": "file:../publish/dist/package" after building the package:
cd src npm run build.debug
but I get this error
Cannot read property 'addSource' of undefined
or map not loaded
@alexonozor iOS or Android? I've only updated Android so far.