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

Remove preview image integration #9657

Merged
merged 1 commit into from
Aug 1, 2017
Merged

Remove preview image integration #9657

merged 1 commit into from
Aug 1, 2017

Conversation

tobrun
Copy link
Member

@tobrun tobrun commented Aug 1, 2017

This PR removes integration for our preview thumbnail in the layout editor of Android Studio.
Current preview image is packaged within the aar and if not filtered out by proguard. This image will be packaged with the apk of end users. This is not ideal and while we wait for a stable Android Studio 3.0 (that will introduce a the @sample annotation), we should remove this integration for now.

Refs #9480

@tobrun tobrun added the Android Mapbox Maps SDK for Android label Aug 1, 2017
@tobrun tobrun added this to the android-v5.1.2 milestone Aug 1, 2017
@tobrun tobrun self-assigned this Aug 1, 2017
@tobrun tobrun requested a review from zugaldia August 1, 2017 06:51
@tobrun tobrun force-pushed the tvn-remove-preview branch from b1f661f to 57eb2e7 Compare August 1, 2017 07:24
@tobrun tobrun force-pushed the tvn-remove-preview branch from 57eb2e7 to 82d16b7 Compare August 1, 2017 07:44
@tobrun tobrun changed the title [android] - remove preview image integration Remove preview image integration Aug 1, 2017
Copy link
Member

@zugaldia zugaldia left a comment

Choose a reason for hiding this comment

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

This is not ideal and while we wait for a stable Android Studio 3.0 (that will introduce a the @sample annotation), we should remove this integration for now.

Could you ticket the follow up work required to bring this pic back once AS3 gets released?

@tobrun
Copy link
Member Author

tobrun commented Aug 1, 2017

Could you ticket the follow up work required to bring this pic back once AS3 gets released?

I was planning to update the original ticket in #9480, with latest status (it already contains links to@sample features).

@tobrun
Copy link
Member Author

tobrun commented Aug 1, 2017

content of #9480 has been updated to reflect the state of this PR, going to merge.

@tobrun tobrun merged commit de46df2 into master Aug 1, 2017
@tobrun tobrun deleted the tvn-remove-preview branch August 1, 2017 13:25
@tobrun tobrun mentioned this pull request Aug 2, 2017
19 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants