rnystrom / RNBlurModalView

Add depth to your alerts.
MIT License
830 stars 143 forks source link

Smoother blur and fixes unused var warning #11

Closed Air-Craft closed 11 years ago

Air-Craft commented 11 years ago

Adds another blur with half the box size and composites with alpha blending. A little more thought and a couple additional steps could make a really nice Gaussian approximation.

Thanks for the big head start!

rnystrom commented 11 years ago

This is really gorgeous. Great work on the blurring. The only problem I have is that the blurred layer is a bit brighter than the original layer. Thoughts?

Air-Craft commented 11 years ago

Hmm... Try now. I've removed the alpha compositing (don't need it anyway really) which has solved the lightening issue...

rnystrom commented 11 years ago

Kickass. And I definitely learned a bit from this PR. Thanks a ton!