Add Fill API support #49
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@toburn Sorry to bother, but have you had a chance to make any progress with this? And also, is there a chance the same functionality is going to be implemented for iOS, too, or do you not have access to necessary hardware and/or software to do that yourself? |
|
@skreborn will try to get to finishing this PR ;) he idea for annotations API on both platforms is to expose a minimal API that works on both. |
Closed
Closed
|
@tobrun would you mind if i build upon this branch? Happy to rebase it and the iOS implementation. |
|
@TimothySealy that would be great though I have a updated version of this branch on another machine. I will push try pushing that updated branch asap. |
…ak consecutive fills though.
|
A status update on this PR. Currently we have a roughly working version. There are some interesting issue still:
Aside from the fill example page I've added fills to the query rendered feature in the map ui page. Now when you filter on zoo's and you tap on an area which is of type zoo the fill for that area is shown. |
Might this be related to mapbox/mapbox-plugins-android#921? |
|
@tobrun @TimothySealy |
IcyTv
added a commit
to IcyTv/flutter-mapbox-gl
that referenced
this pull request
Nov 15, 2020
* Expose CompassViewPosition flutter-mapbox-gl#344 (flutter-mapbox-gl#346) Co-authored-by: emre.yalcin <emre.yalcin@netcad.com.tr> * Readme: location features on Android (flutter-mapbox-gl#342) * Update CONTRIBUTING.md (flutter-mapbox-gl#366) * Release 0.8.0 (flutter-mapbox-gl#390) * [release] update CHANGELOG.md for release of v0.8.0 * [release] update version numbers to v0.8.0 * Update CONTRIBUTING.md (flutter-mapbox-gl#401) Fix stale pull, issues & changelog urls in the contributing guide. * Fix data parameter for addLine and addCircle (flutter-mapbox-gl#388) * Split padding values in CameraUpdate.newLatLngBounds() (flutter-mapbox-gl#382) * Split padding values in CameraUpdate.newLatLngBounds() * Remove old unused code Co-authored-by: Tobrun <tobrun.van.nuland@gmail.com> * Re-enable attribution on android (flutter-mapbox-gl#383) * Upgrade Android annotation plugin to v0.9 (flutter-mapbox-gl#381) * web: ignore myLocationTrackingMode if myLocationEnabled is false (flutter-mapbox-gl#363) * Add methods to access projection (flutter-mapbox-gl#380) * remove bitmap; add projection access * Replace ScreenLocation with Point; expand iOS implementation * fix iOS with guard let * iOS: cast to NSObject * fix typo * round result of toScreenLocation() * Revert "round result of toScreenLocation()" This reverts commit 838726a. * Docs: document rounding behaviour * Add Fill API support (flutter-mapbox-gl#49) * [flutter] [android] - add fill support * Resolved merge conflict. * A first working version for ios (after some extensive rebasing). * Minor cleanup * Minor cleanup. * Fix broken build Android. * A working version for Android. * Minor cleanup. * Added fill pattern example. Works on Android not on iOS. Seems to break consecutive fills though. * For the first queried feature (when filter is set) create a fill. * Fix lint issue (unused method). * Updated code formatting. * Added interior polygon to iOS. * [docs] update readme support table * fixup Co-authored-by: Timothy Sealy <timothy.sealy@gmail.com> * Listen to OnUserLocationUpdated to provide user location to app (flutter-mapbox-gl#237) * Listen to OnUserLocationUpdated to provide user location to app While the `myLocationEnabled` property is set to `true`, this method is called whenever a new location update is received by the map view. iOS only, needs Android. I did check that the location properties carried here are also provided in Android's [Location][1] object. [1]: https://developer.android.com/reference/android/location/Location * add android, web; fix conflicts Co-authored-by: m0nac0 <58807793+m0nac0@users.noreply.github.com> Co-authored-by: Tobrun <tobrun.van.nuland@gmail.com> * fix: correct bug on android where checking on activity lifecycles that were disposed (flutter-mapbox-gl#266) Co-authored-by: leo cornillon <leo.cornillon.dev@gmail.com> Co-authored-by: m0nac0 <58807793+m0nac0@users.noreply.github.com> * Add support for custom font stack in symbol options (flutter-mapbox-gl#359) * fix memory leak caused by strong self reference (flutter-mapbox-gl#370) * Basic ImageSource Support (flutter-mapbox-gl#409) * Introduce LatLngQuad Introduce the LatLngQuad object which will be useful to pass in all the required parameters to the addSource() method we will define later. * Introduce addSource() Method Add the addSource(..) method to the mapbox_gl_platform_interface.dart * Add addSource MethodChannel * Place ImageSource Android - Introduce the PlaceSource page as a playground - Add 'addImageSource', 'removeImageSource', 'addLayer' & 'removeLayer' apis - Implement Android platform interface * iOS ImageSource Implementation Implement addImageSource, removeImageSource, addLayer & removeLayer on iOS. * Fix iOS CoordinateQuad Mapping Co-authored-by: Tobrun <tobrun.van.nuland@gmail.com> * Get meters per pixel at latitude (flutter-mapbox-gl#416) * fix git refferences * fix git refferences * implementation of getMetersPerPixelAtLatitude * getMetersPerPixelAtLatitude * fix refference paths * Android implementation and Example updated. * added comments to getMetersPerPixelAtLatitude method * IOS implementation * Removed modified lines from pubspec.yaml files * web implementation Co-authored-by: Tobrun <tobrun.van.nuland@gmail.com> * fix onStyleLoadedCallback (flutter-mapbox-gl#418) * fix onStyleLoadedCallback * fix onStyleLoadedCallback called before onMapCreated Co-authored-by: Tobrun <tobrun.van.nuland@gmail.com> * Release 0.9.0 * update version numbers for v0.9.0 release * fix ios onSymbolTapped Co-authored-by: emreuguryalcintr <50848628+emreuguryalcintr@users.noreply.github.com> Co-authored-by: emre.yalcin <emre.yalcin@netcad.com.tr> Co-authored-by: m0nac0 <58807793+m0nac0@users.noreply.github.com> Co-authored-by: Tobrun <tobrun.van.nuland@gmail.com> Co-authored-by: cuberob <robdeknegt@gmail.com> Co-authored-by: qbouchat <48958612+qbouchat@users.noreply.github.com> Co-authored-by: Timothy Sealy <timothy.sealy@gmail.com> Co-authored-by: Nathan <nathan@transit.app> Co-authored-by: Leo Cornillon <corle.cugly@gmail.com> Co-authored-by: leo cornillon <leo.cornillon.dev@gmail.com> Co-authored-by: Philip Lindberg <philip.c.lindberg@gmail.com> Co-authored-by: Thomas Kröniger <30893720+thirteenthstep@users.noreply.github.com> Co-authored-by: cuberob <git@cuberob.com> Co-authored-by: GULERTOLGA <gulertolga@gmail.com> Co-authored-by: Andrea Valenzano <andr3a689@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
WIP, this PR adds support for the fill annotation type.