Skip to content

Conversation

@svendvd
Copy link
Contributor

@svendvd svendvd commented Dec 17, 2021

No description provided.

@svendvd svendvd requested a review from prempalsingh December 17, 2021 14:43
}
}

}
Copy link
Member

Choose a reason for hiding this comment

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

could you use the source code from the latest deployed docs? That has better handling for gallery intent, doesn't use the deprecated getExternalStorageDirectory, and also doesn't have un-needed permission handling.

android:layout_height="match_parent"
android:layout_weight="25"/>
</LinearLayout>
</RelativeLayout> No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

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

not needed in this sample?

<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE"/>
<uses-permission android:name="android.permission.INTERNET"/>

<protected-broadcast android:name="android.intent.action.MEDIA_MOUNTED" />
Copy link
Member

Choose a reason for hiding this comment

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

why is this broadcast needed? 😅


// We hijack internalLoadBitmapCache to know when it's time to load the cache
override fun internalLoadBitmapCache(pixelSize: Long, sync: Boolean): Boolean {
if (!isClickCacheLoading && clickAreaCache == BitmapFactoryUtils.NOTHING_BITMAP) {
Copy link
Member

Choose a reason for hiding this comment

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

can we add a check for !sync here since this is not needed in the case of exporting..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants