[FR] Expand return type on imperative APIs

Currently all of the imperative APIs return a Drawable, which is great for drawing but it would be handy to know more information about the load itself. Things like: where did the image come from? (cache? network?)

I imagine something like the transition API:

enum class Source { DISK, MEMORY_CACHE, NETWORK }

sealed class LoadResult {
    class Success(
        val drawable: Drawable,
        val source: Source
    ) : LoadResult()

    class Error(
        val drawable: Drawable?
    ) : LoadResult()
}
    
fun Coil.getResult(data: Any): LoadResult

This would be super useful for when Coil is being used in outside of View environments 🚀

WDYT?

[colinrtwhite]

@chrisbanes Ended up changing my mind on this 🙂. Implemented here: https://github.com/coil-kt/coil/pull/349 and will be part of the next release.

[colinrtwhite]

It’s probably simpler than that. See here for a quick impl.

Oh neat I didn’t know about OnChildPositioned.

Basically replicating what you did for Transition but for things which don’t use Drawables

Ah I see what you mean. My preference would be to integrate Coil’s Transition with the Compose Transition class if that’s possible. However, that doesn’t look likely at the moment… Going to play around with Compose more to improve my understanding.