Dumb question: How to make the compose Preview rendering works?

[duongphuhiep]

It seems that val playerState = rememberVideoPlayerState() is not previewable.

Do we have some kind of

val playerState = PreviewableVideoPlayerState()

Or Should I make a abstract wrapper in order to make thing previewable?

// commonMain  
interface VideoPlayerStateInterface {  
    val isPlaying: Boolean  
}  
  
// Real implementation wrapper  
class RealVideoPlayerStateWrapper(  
    private val playerState: VideoPlayerState  
) : VideoPlayerStateInterface {  
    override val isPlaying = playerState.isPlaying 
    ...
}  
  
// Preview implementation  
data class PreviewVideoPlayerState(  
    override val isPlaying: Boolean = false,  
    ...
) : VideoPlayerStateInterface  
  
// My composable now uses the interface  
@Composable  
fun KdroidVideoPlayer(playerState: VideoPlayerStateInterface) {  
    // UI implementation  
}

[duongphuhiep]

Update: The wrapper won't help the Preview rendering because the VideoPlayerSurface would still require a concret non nullable playerState -> VideoPlayerSurface is not previewable whatever we do.

// not-previewable
VideoPlayerSurface(
        playerState = playerState,
        modifier = modifier
    ) {
}

If VideoPlayerState were an interface then it will be easy to mock it in preview... I wonder why it isn't?
it could even be a AutoClosable interface: interface IVideoPlayerState: AutoCloseable {}

[duongphuhiep]

Thanks for your reactivity.

The changes might prevents the Preview from crashing, but it isn't real support for previewing (or testing).

I made a quick MR to show the idea (not fully tested yet => so feel free to reject it)

#147

[kdroidFilter]

I’m sorry, I’m really overbooked at the moment. Thank you very much for your efforts. I’ll try to make some time to fix the most critical bugs, and I’ll check this tonight.

I originally developed this library mainly for Compose Desktop, so I didn’t really pay attention to previews, which weren’t working very well at the time I wrote the library.