Skip to content
This repository was archived by the owner on Aug 8, 2023. It is now read-only.

WIP GLSurfaceView#8031

Closed
tobrun wants to merge 6 commits into
masterfrom
tvn-wip-state-keeping-sync
Closed

WIP GLSurfaceView#8031
tobrun wants to merge 6 commits into
masterfrom
tvn-wip-state-keeping-sync

Conversation

@tobrun
Copy link
Copy Markdown
Member

@tobrun tobrun commented Feb 13, 2017

This is a WIP PR that covers migration to GLSurfaceView.

  • use GLSurfaceView instead of SurfaceView
  • move rendering to render thread
  • invoke all nativeMapView methods on the render thread
  • provide callback interfaces for getters

Todo:

  • use Android SDK animators for gestures and camera animations.
  • use promisses/futures for setters
  • make components as transform, annotationmanager thread safe
  • handle exceptions from render thread on UI thread (eg. NoSuchLayerException)
  • bugs & crashes

cc @ivovandongen @mapbox/android

@tobrun tobrun added the Android Mapbox Maps SDK for Android label Feb 13, 2017
@tobrun tobrun added this to the android-v5.0.0 milestone Feb 13, 2017
@tobrun tobrun requested a review from ivovandongen February 13, 2017 07:22
@mention-bot
Copy link
Copy Markdown

@tobrun, thanks for your PR! By analyzing this pull request, we identified @jfirebaugh, @zugaldia and @1ec5 to be potential reviewers.

@tobrun tobrun mentioned this pull request Feb 13, 2017
7 tasks
* <p>
* The {@link com.mapbox.mapboxsdk.maps.MapboxMap.MarkerViewAdapter#onDeselect(MarkerView, View)}
* will be called to execute an animation.
* will be called to executeOnRenderThread an animation.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like a search-and-replace made some unintended changes.

@1ec5
Copy link
Copy Markdown
Contributor

1ec5 commented Feb 22, 2017

use Android SDK animators for gestures and camera animations.

Tracking this refactoring in #8175, in case there’s an opportunity to implement it separately ahead of this PR.

@tobrun tobrun removed this from the android-v5.0.0 milestone Mar 6, 2017
@ivovandongen
Copy link
Copy Markdown
Contributor

Closing in light of #8820

@jfirebaugh jfirebaugh deleted the tvn-wip-state-keeping-sync branch October 24, 2017 20:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Android Mapbox Maps SDK for Android ⚠️ DO NOT MERGE Work in progress, proof of concept, or on hold

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants