Wrong activity background blurring effect

[luongvo]

Please include:

1. Library version: version-2.0.0
2. Device and OS version: all API versions, tested on API 23 and 30
3. Detailed steps to reproduce the issue

• Added a new transparent fragment
• Added a android:windowBackground attr
• Checkout the blurring effect on topBlurView on transparent fragment

4. XML layout and code for BlurView setup: Reproduce commit here luongvo@6835fd2

5. Stacktrace in case of a crash: None

Expected: set android:background="@drawable/lines" in activity_main.xml could produce an expected result

expected.mp4

Actual: using <item name="android:windowBackground">@drawable/lines</item> in AppTheme and .setFrameClearDrawable(windowBackground) set in MainActivity does not work properly.

actual.mp4

[Dimezis]

Apparently, there's some weird bug with android:windowBackground as an image (BitmapDrawable).
I'll see what I can do about it, but be aware that even regardless of this bug, your Expected result is probably impossible with setFrameClearDrawable, because the drawable doesn't have a trackable position on the screen.

That means, that even if I fix the bug, the bottom BlurView won't be able to properly blur the bottom of this drawable.

To fix this, you can either pass the (ViewGroup) getWindow().getDecorView() as a rootView, or set the background in activity_main.xml as you did.

[luongvo]

@Dimezis I tried to apply using getWindow().getDecorView() but it does not work.

This lib https://github.com/mmin18/RealtimeBlurView using the same approach but it works properly, here is the POC luongvo/RealtimeBlurView@b87607a. The bad thing is https://github.com/mmin18/RealtimeBlurView does not support latest API RenderEffect from API 31 and discontinued.

RealtimeBlurView-poc.mp4

I think even there are some issues, we should support getWindow().getDecorView() directly from this nice library instead.

[Dimezis]

I tried to apply using getWindow().getDecorView() but it does not work.

Can you post a commit with this change?
It works fine for me on your branch

[luongvo]

@Dimezis I made a draft PR here, please check it out #179 🙏

The update from the PR above still does not work:

Actual:

Expected:

[luongvo]

@Dimezis I think this logic could help https://github.com/mmin18/RealtimeBlurView/blob/master/library/src/com/github/mmin18/widget/RealtimeBlurView.java#L245-L261 🤔

[Dimezis]

Responded - #179 (comment)