nativescript-mapbox icon indicating copy to clipboard operation
nativescript-mapbox copied to clipboard

Added above/below options to addLayer/addPolygon/addPolyline

Open lochstar opened this issue 5 years ago • 13 comments

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.

lochstar avatar Mar 22 '19 03:03 lochstar

Looks like a reasonably nice solution to the problems reported 👌 was looking at doing this too

Codex- avatar Mar 22 '19 11:03 Codex-

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 readonlys with nothing after them).

Codex- avatar Mar 26 '19 21:03 Codex-

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

Codex- avatar Mar 30 '19 03:03 Codex-

Tested these changes on my branch, all seems 👌

Codex- avatar Apr 09 '19 00:04 Codex-

@lochstar my PR was accepted, so you'll need to resolve / rebase probably.

Codex- avatar Apr 30 '19 21:04 Codex-

@Codex- Sorted!

lochstar avatar Apr 30 '19 22:04 lochstar

@EddyVerbruggen Can we get this merged? :)

lochstar avatar May 29 '19 01:05 lochstar

Probably won't until the failing build is resolved?

Codex- avatar May 29 '19 01:05 Codex-

Thanks @Codex-

lochstar avatar May 29 '19 05:05 lochstar

How do I run the branch containing the addsource and addLayer?

alexonozor avatar Sep 22 '20 12:09 alexonozor

@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

lochstar avatar Sep 22 '20 21:09 lochstar

@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 avatar Sep 25 '20 08:09 alexonozor

@alexonozor iOS or Android? I've only updated Android so far.

lochstar avatar Oct 01 '20 07:10 lochstar