Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add bounds to overlay view #206

Closed
wants to merge 4 commits into from
Closed

Add bounds to overlay view #206

wants to merge 4 commits into from

Conversation

uriklar
Copy link

@uriklar uriklar commented Feb 14, 2016

No description provided.

@ilyagelman
Copy link

@tomchentw This is a very useful one, could you please consider merging?

@tomchentw tomchentw closed this in dcdfefa Feb 21, 2016
@tomchentw
Copy link
Owner

Thanks! Released v4.9.0

@tomchentw
Copy link
Owner

Feel free to open up a PR to add an example to example/gh-pages!

@uriklar
Copy link
Author

uriklar commented Feb 21, 2016

Hi @tomchentw . Thanks for merging! I actually did add an example to gh-pages. It's in:
examples/gh-pages/scripts/components/basics/OverlayView.js

@tomchentw
Copy link
Owner

@uriklar yeah, but I thought you just added a comment?

tomchentw added a commit that referenced this pull request Feb 21, 2016
@tomchentw
Copy link
Owner

I found that changing from cDU to cWRP will fail the test case. I just reverted and released a new version v4.9.1.

Please raise up a PR of this issue if you think it's a better way to go and have fixed the failing test cases, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants