OnTouchEventListener stop working after adding inner Fragments on Activity

Hi, I’m trying to use OnTouchEventListener like this:

class PingDispatcherLifecycleCallback : Application.ActivityLifecycleCallbacks {
    private val pingDispatcher = OnTouchEventListener { motionEvent ->
        if (motionEvent.action == KeyEvent.ACTION_DOWN) {
            Log.d("ping", "send")
        }
    }

    override fun onActivityResumed(activity: Activity) {
        activity.window.touchEventInterceptors += pingDispatcher
    }

    override fun onActivityPaused(activity: Activity) {
        activity.window.touchEventInterceptors -= pingDispatcher
    }
    ...
}

if put Fragment into Activity and add child Fragment into root Fragment It stop to receive onTouch events

it become works after adding this callback into PingDispatcherLifecycleCallback in onActivityResumed callback

class PingDispatcherLifecycleCallback : Application.ActivityLifecycleCallbacks {
    ...
    private val fragmentOnAttachListener = object : FragmentManager.FragmentLifecycleCallbacks() {
        override fun onFragmentResumed(fm: FragmentManager, f: Fragment) {
            window.touchEventInterceptors -= pingDispatcher
            window.touchEventInterceptors += pingDispatcher
        }
    }

    override fun onActivityResumed(activity: Activity) {
        supportFragmentManager.registerFragmentLifecycleCallbacks(fragmentOnAttachListener, true)
    }
    ...
}

is it bug or legal behaviour of Android?

[pyricau]

Closing, partially fixed by https://github.com/square/curtains/pull/18 (though really the fix is in appcompat)

[pyricau]

I do expect the callback to be replaced, but the key difference / expectation is that I expect it to be chained, ie I expect ToolbarActionBar$ToolbarCallbackWrapper to wrap AppCompatDelegateImpl.AppCompatWindowCallback and then AppCompatDelegateImpl.AppCompatWindowCallback to wrap curtains.internal.WindowCallbackWrapper

However, as I was saying, this won’t work with the current release. The next release, after #18 is merged, will make that change work.