-
Notifications
You must be signed in to change notification settings - Fork 502
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Create a static map snapshot #1076
Create a static map snapshot #1076
Conversation
@hungtrn75 im quite busy right now - will take some time until i can review this |
hoping to get this feature merge. |
Looks good to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also run get formatting? checkout flutter_ci.yml
case "snapshot#takeSnap": | ||
{ | ||
FileSource.getInstance(context).activate(); | ||
MapSnapshotter.Options snapShotOptions = new MapSnapshotter.Options((int) call.argument("width"), (int) call.argument("height")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this could be final?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes. this is final
mapbox_gl_platform_interface/lib/src/mapbox_gl_platform_interface.dart
Outdated
Show resolved
Hide resolved
* Add makefile for formatting (flutter-mapbox-gl#1181) * refactor and fix ci the file * add makefile for formatting and installing formatting tools * Create a static map snapshot (flutter-mapbox-gl#1076) * android impl * take snapshot ios * config for each platform interface * take snapshot in ios * take snap in android && example * android hybrid composition * ios test * test with android hybrid composition * Update README.md * use JPG instead of PNG with writeToDisk option & remove unused funtions * render example result with base64 option * remove team ID in example * iOS: use JPG instead of PNG * rename funtion * test flutter ci * test ci * ci: check swift formatting * ci: test check java formatting * ci: check java formatting * ci: test check java formatting * document for take snapshot feature * revert ci config * migration: jpeg with base64 option * docs: web support * feat: web support with base64 option * ci: test github ci * lint: ignore unnecessary_import * ci: reverse config Co-authored-by: Felix Horvat <felix.horvat@ocell.io> Co-authored-by: Hung Tran <32390731+hungtrn75@users.noreply.github.com>
This reverts commit 3e490a8.
PR for #957