feat: android blurview library updated#598
feat: android blurview library updated#598dopplerDistortion wants to merge 3 commits intomargelo:masterfrom
Conversation
|
@dopplerDistortion Hey! Awesome that you started a PR, but its not gonna help on the problem by itself. Here is a patch file I created which has a lot of tweaks to bring the look of the iOS and Android versions closer to each other: @react-native-community+blur+4.3.2.patch Can you please apply this to your repository somehow? |
|
I hope when its done, then @Kureev can help us merge it. |
|
@stemyke Hey! I tried but, patch file has some auto generated changes and files inside. If you can send the diff your changes instead of patch file it would be much more useful. |
|
Sure! Here are all the files from the package including the modified ones: |
|
@dopplerDistortion Oh no! You are right, I did not understand why was the BlurViewManager separated from its implementation, now I get it, because it was based on the RN architecture and it is different. Can you please restore BlurViewManager to its original place, but only with the tweaked and removed properties? And please do that also in the newarch if you can. I feel like the BlurViewManagerImpl is still unnecessary because now I am doing the necessary calculations inside the BlurView. Edit: I only use oldarch in my project and I did the changes directly in node_modules and thats why I did not notice it. |
|
@stemyke it is restored to original place. But I don't have enough understanding about new arch. Maybe you can send their changes after checking it. |
Android BlurView library updated with @stemyke's PR Dimezis/BlurView#201
closes #511